Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 533: Update developer documentation #541

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

dogle-scottlogic
Copy link
Contributor

@dogle-scottlogic dogle-scottlogic commented Nov 15, 2024

Update the developer document to be a more complete contributors guide.

Resolves issue #533

@dogle-scottlogic dogle-scottlogic requested a review from a team as a code owner November 15, 2024 09:57
@dogle-scottlogic dogle-scottlogic added the community structure Work related to the Community Structure WG label Nov 15, 2024
@dogle-scottlogic dogle-scottlogic requested a review from a team as a code owner November 15, 2024 10:27
@dogle-scottlogic dogle-scottlogic requested a review from a team November 15, 2024 13:15
Copy link
Contributor

@damienjburks damienjburks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Honestly, I think this should be at the root of the entire project called CONTRIBUTING.md. It would be easier for folks to find it. Thoughts on this @dogle-scottlogic?

@dogle-scottlogic
Copy link
Contributor Author

@damienjburks - we could do, the other option is to link to this doc from the root README.md so there's just a single doc at the root level to get started with the project but happy either way?

@damienjburks
Copy link
Contributor

@dogle-scottlogic I think the better approach would be to move it out to the root as CONTRIBUTING.md. Let's do that.

damienjburks
damienjburks previously approved these changes Nov 18, 2024
@dogle-scottlogic dogle-scottlogic force-pushed the CCC-533-developer-guide branch 3 times, most recently from b749f97 to 998fc81 Compare November 19, 2024 12:28
@dogle-scottlogic dogle-scottlogic requested a review from a team as a code owner November 19, 2024 13:57
damienjburks
damienjburks previously approved these changes Nov 19, 2024
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that this change will also need to remove .github/CONTRIBUTING.md, which is linked under the description when folks create a PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community structure Work related to the Community Structure WG
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants