-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
chore: Add issue section to contributing guide #29
Conversation
Awesome @d-a-ve Ii will have Geekiedj review this. |
Alright, sounds good @DominusKelvin |
Hi @d-a-ve, thank you for your contribution. Instead of using [chore], [feat], or [fix] prefixes in the issue title, it's recommended to use labels for categorization. These prefixes are commonly used in commit messages or PR titles, not when creating an issue. |
Okay, so labels such as [bug], [doc], etc. Or what kind of labels do you have in mind @geekiedj |
yeah labels such as "good first issues, enhancement, etc. You can find them on the right of this page or better still, you could check the issues tab to understand what I'm talking about |
I have seen them; bug, enhancement, good-first-issue, wont-fix etc. So these should be added to the issue, alright. Will the person be the one to select the issue label that fits or that will be done by the maintainer? @DominusKelvin @geekiedj |
I assume you're trying to add a Does that make sense? @d-a-ve |
Yes this is great, I understand. So the person picks the labels that fits the issue. I will make a pull request later in the day for the changes. |
@geekiedj What do you think? It seems the person that creates an issue cannot assign the labels themselves...unless I do not know how to do it |
Oh! Just checked now, that's true. It has to do with some permission things. If that be the case, you could take that option out and just make a PR. |
Okay, I didn't add it to the last commit I sent, I was waiting for your feedback. Can you check the issue section I created in the last commit I made? So we can be sure it is okay before merging. |
1 similar comment
Okay, I didn't add it to the last commit I sent, I was waiting for your feedback. Can you check the issue section I created in the last commit I made? So we can be sure it is okay before merging. |
Okay, I didn't add it to the last commit I sent, I was waiting for your feedback. Can you check the issue section I created in the last commit I made? So we can be sure it is okay before merging. Here is the commit |
I thought I replied already.Apologies. I think this looks good. Let's see what @DominusKelvin thinks? |
Alright. @DominusKelvin What do you think? |
CONTRIBUTING.md
Outdated
## Creating an Issue | ||
Before **creating** an issue, please follow these steps: | ||
|
||
1. Search existing issues before creating a new issue (has someone created an issue already). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, can we rephrase it to do this @d-a-ve 👇🏾
Before creating a new issue, it is advisable to search for any existing issues that may have already been created.
CONTRIBUTING.md
Outdated
1. Search existing issues before creating a new issue (has someone created an issue already). | ||
2. If it doesn't exist, create an issue giving as much context and information as possible in the description. | ||
> Images and videos can be added to the description to provide more information. | ||
3. Use an appropriate format for the title: `[bug]`, `[documentation]`, `[help wanted]`, or `[good first issue]` etc, followed by a concise title description. For example, an issue that relates to The Boring Tech Stack documentation could have a title like `documentation: Add issue section to Contributing guidelines`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the example didn't reference the guide, so can it be rephrased and also for prefix, let's go with bug:, docs:, help wanted: etc
Hence you example will look like docs: Add issue section to Contributing guidelines
FYI, the angle bracket syntax is intended for PRs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't know the angle brackets are for PRs, alright.
I will make the necessary changes and let you know
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@d-a-ve this looks good. I pointed out a couple of things. Once they are fixed, I'll proceed to merge this. Thanks :)
@d-a-ve thanks for working on this, I left a couple of suggestions for you :) |
@DominusKelvin I have made the changes. How is it now? |
Hi @DominusKelvin please have you been able to check the change i did with the new commit? |
Closes #24