Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

Organize GitHub Labels #610

Closed
carolynvs opened this issue May 19, 2017 · 7 comments
Closed

Organize GitHub Labels #610

carolynvs opened this issue May 19, 2017 · 7 comments

Comments

@carolynvs
Copy link
Collaborator

Once we have defined the various components of dep in #609, create labels for each area so that issues can be flagged with their impacted areas.

@carolynvs
Copy link
Collaborator Author

@sdboyer I created the following labels. Let me know what you think, and if I get a 👍 I'll run through and flag existing issues with them.

  • area: init
  • area: ensure
  • area: status
  • area: gps
  • area: shared

@darkowlzz
Copy link
Collaborator

I like the above area labels. I would like to add one more "area: testing" or something similar for things that are related to overall tests, integration tests. Or does it makes more sense to put them under shared? 🤔 Less number of labels :)

Not exactly related to impacted area labels but maybe it would be helpful to also have general PR life cycle process labels to show the status of PRs. I came across some projects where there are defined phases of a PR life cycle, like design-review, code-review, docs-review and then merge. This ensures that we don't miss things like updating docs when something is changed. A clearly defined process would also help contributors to understand, prepare and submit good PRs.

Thoughts?

@sdboyer
Copy link
Member

sdboyer commented May 31, 2017

@carolynvs I think the area labels are mostly good, though I might omit "area: shared", as at least for now, i think we can treat that as implicit - if it's not one of the other areas, then it's shared. That and/or if it's in multiple areas, then apply the label for each of the areas it touches.

I would like to add one more "area: testing" or something similar for things that are related to overall tests, integration tests.

To the extent that these labels are supposed to reflect areas of responsibility for subsystem maintainers within the project, I'd say that the tests are implicitly included in the corresponding subsystem that they test.

We could probably use a test label, but i think i's orthogonal to the "area" idea.

Not exactly related to impacted area labels but maybe it would be helpful to also have general PR life cycle process labels to show the status of PRs.

Definitely valuable, and yes, we could do this with labels. However, i've been using zenhub a bit with dep, though, and i'd say it's probably preferable that we express this kind of information using their pipelines.

@carolynvs
Copy link
Collaborator Author

Yup, the goal was just to make it easier for various maintainers to keep up with issues in their area. I've removed area: shared and will apply the labels today.

@darkowlzz
Copy link
Collaborator

Just checked zenhub and 😊 to be able to see more stuff in another dimension (zenhub-github 😬).

@carolynvs
Copy link
Collaborator Author

Closing because it sounds like everyone is happy. 😇

@carolynvs
Copy link
Collaborator Author

I've done a quick pass through our issues and applied those labels. If I flagged something incorrectly, feel free to tweak!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants