-
Notifications
You must be signed in to change notification settings - Fork 18
Standard labels #72
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
Standard labels #72
Conversation
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.
This is awesome! Love the addition of the logger.
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.
@dhruvkb really great work, and I like the new logging system. :)
I hadn't reviewed the label descriptions earlier, I added a bunch of comments to them. Basically, we want the shortest, simplest, label descriptions (that are accurate) so that they're both quick/easy to read and more accessible to non-native English speakers. Please feel free to make other changes as well, I only commented on some of them, and even those may be able to be simplified.
Description
This PR adds code that updates the standard labels used across all of CC's applications. Existing labels will not be affected. You can use the new labels in new issues for now, old issues will be automatically moved later.
Oh and it adds a new logging framework that
print()
statementsChecklist
Update index.md
).main
ormaster
).visible errors.
Developer Certificate of Origin
Developer Certificate of Origin