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

ci: bug triage #982

Merged
merged 6 commits into from
Aug 30, 2021
Merged

ci: bug triage #982

merged 6 commits into from
Aug 30, 2021

Conversation

lionel-nj
Copy link
Contributor

@lionel-nj lionel-nj commented Aug 30, 2021

Summary:

This PR provides support to programmatically create a card in bug triage board when a new bug is reported (i.e. an issue is labelled with bug).

Expected behavior:

If an issue is labelled bug it should be added to the board in the column "Requires investigation".
Also an automated message should appear in the issue:

"Thank you for your reporting a bug. The issue has been placed in triage, the MobilityData team will follow-up on it."

See #983:

Capture d’écran, le 2021-08-30 à 12 12 54

Capture d’écran, le 2021-08-30 à 12 12 31

Please make sure these boxes are checked before submitting your pull request - thanks!

  • [ ] Run the unit tests with gradle test to make sure you didn't break anything
  • Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
  • [ ] Linked all relevant issues
  • Include screenshot(s) showing how this pull request works and fixes the issue(s)

@lionel-nj lionel-nj self-assigned this Aug 30, 2021
@lionel-nj lionel-nj marked this pull request as ready for review August 30, 2021 15:15
Copy link
Member

@barbeau barbeau left a comment

Choose a reason for hiding this comment

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

Looks like this does what it says it does :). FWIW, I'm used to managing priority and triaging issues differently via labels (see https://github.com/googlemaps/android-maps-utils for example), but if you prefer this it works 👍.

@lionel-nj
Copy link
Contributor Author

Looks like this does what it says it does :). FWIW, I'm used to managing priority and triaging issues differently via labels (see https://github.com/googlemaps/android-maps-utils for example), but if you prefer this it works 👍.

Thanks for reviewing. That is something we could do in the future provided we come up with a clear definition for each priority level @isabelle-dr.

@lionel-nj lionel-nj merged commit b8389b6 into master Aug 30, 2021
@lionel-nj lionel-nj deleted the ci/bug-triage branch August 30, 2021 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants