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

Tailor GHA message to improve workflow and individual responsibility when creating issues #2127

Closed
3 of 5 tasks
Aveline-art opened this issue Aug 18, 2021 · 12 comments · Fixed by #2578
Closed
3 of 5 tasks
Assignees
Labels
Complexity: Small Take this type of issues after the successful merge of your second good first issue Feature: Board/GitHub Maintenance Project board maintenance that we have to do repeatedly role: back end/devOps Tasks for back-end developers size: missing Status: Updated No blockers and update is ready for review

Comments

@Aveline-art
Copy link
Member

Aveline-art commented Aug 18, 2021

Overview

As a developer, I want to have the messages posted by the GHA-bot to motivate proper behavior to ensure a smooth team workflow. For this issue, we need to tailor the message that an issue creator will see when one or more feature, role, or size labels are missing upon new issue creation.

Action Items

Product

  • Review the message from add-labels-template.md and no-labels-template.md, to ensure that it is optimal, and motivates the right behavior.
  • If the message is fine, this issue can be closed.
  • If not, post a new message in the comments below with the new message. Be sure to label them so that we can tell which new message is for which file.
  • Replace the 'role:product' label with 'role: backend' if the below items are needed. Else, close the issue.

Developer

  • Locate the new messages posted by the PM team, and edit them as necessary in the md files noted below.

Resources/Instructions

no-labels-template.md
add-labels-template.md

Currently, the message from no-labels-template.md is displayed when an issue is created with all the required labels. However, the add-labels-template.md is unused because the message might not be optimal. Specifically, the current message would motivate adding the missing labels, even if the issue creator does not have the prerequisite expertise to do so (for ex, a developer might improperly add labels for a design issue). Also, since this message pops up when an issue is created, should we leverage this message to include, maybe, other items the issue creator needs to review besides labels (for example, checking that their overview or action items are well formed).

@Aveline-art Aveline-art added the role: product Product Management label Aug 18, 2021
@github-actions github-actions bot added Feature Missing This label means that the issue needs to be linked to a precise feature label. role missing size: missing and removed role: product Product Management labels Aug 18, 2021
@Aveline-art Aveline-art added role: product Product Management and removed role missing labels Aug 18, 2021
@ExperimentsInHonesty

This comment has been minimized.

@ExperimentsInHonesty ExperimentsInHonesty added Feature: Board/GitHub Maintenance Project board maintenance that we have to do repeatedly and removed Feature Missing This label means that the issue needs to be linked to a precise feature label. Ready for Prioritization labels Sep 5, 2021
@ExperimentsInHonesty ExperimentsInHonesty added Complexity: Small Take this type of issues after the successful merge of your second good first issue and removed Ready for Prioritization labels Sep 5, 2021
@ExperimentsInHonesty ExperimentsInHonesty added this to the 08. Team workflow milestone Sep 5, 2021
@ExperimentsInHonesty
Copy link
Member

ExperimentsInHonesty commented Sep 8, 2021

@Aveline-art @abuna1985 @matthewlee626 This message is not current.

It show up even when they have not added labels and the issue is moved automatically now to the New Issue Approval column when its added to the board

Good job adding the required labels to this issue.

Just a friendly reminder to also move the issue into the "New Issue Approval" column under the Project Board. For more info on that, check Github's documentation here. Thanks!``

Product would be happy to take a look at a new draft when you have one.

Moving this back to new issue approval for re writing of action items

@macho-catt
Copy link
Member

macho-catt commented Sep 8, 2021

@ExperimentsInHonesty is there an example issue where the author did not add labels and the post showed up (when it shouldn't)? That way I can troubleshoot.

As for the second line of the post, we can make an issue to remove that or update this issue to remove that.

Thanks

@ExperimentsInHonesty
Copy link
Member

ExperimentsInHonesty commented Sep 8, 2021

@macho-catt What needs to happen is that someone needs to model what happens when you do each of the things below and see where it does not make any sense. This does not need anyone from product to do this, this can be done by front end or back end leads or merge team or anyone who has been around long enough to understand how issues work.

Here is the stuff I would model to see what does not work the way we intend.

  1. What happens when you make an issue and don't add any labels.
  2. What happens if you make an issue and add 1 label.
  3. What happens if you make an issue and add all the required labels (size, feature, role)

put the ready for milestone label on this again when you are ready for me to sign off on the issue

@Aveline-art
Copy link
Member Author

@ExperimentsInHonesty I am not sure what exactly is being asked of us. Would you like us to create some chart or graphic that clears up what the flow of the GHA is?

@macho-catt
Copy link
Member

macho-catt commented Sep 26, 2021

We can discuss this workflow during one of the dev meetings (either Tue or Thu) and get feedback from developers to see which makes sense.
Currently, it works like this:

  1. If I create an issue and I don't add any labels:
    • the three labels for "missing" will be added: "size missing", "feature missing", "role missing"
  2. If I create an issue with only one of the required labels (for example, size: medium):
    • the two other labels for "missing" will be added: "feature missing", "role missing"
  3. If I create an issue and add all required labels:
    • The bot does not add any labels and posts a comment telling the user they added the proper labels.
    • The comment previously mentions the project board, but since that is now automated it has been removed
    • The comment also links to wiki articles for issue creation and labels, but those wiki articles have been postponed

Other things that will need to be brought up with devs:

  1. There is currently an unused message (the code is commented) for when a user forgets to add the proper labels. Due to a discussion with @Aveline-art, we decided to remove it for now. The requirements may have changed now so we can discuss to use this code again.
  2. The current used message for point 3 was initially made because the project board was not automated. Now that it's just text for congratulating the user + links to unfinished wiki articles, is it still relevant? Can we put other things in it?
  3. There is a a fun race condition where if a user creates an issue without any of the proper labels, it will trigger the action. However, if during the time the action is triggering the user adds the proper labels, the github action will remove the labels the user just added and put the three missing labels. The problem: when the github action triggers, the API will only look at the issue's metadata during the creation; it won't look at the metadata of the issue after it's created.

@macho-catt macho-catt added role: back end/devOps Tasks for back-end developers and removed role: product Product Management labels Nov 1, 2021
@macho-catt
Copy link
Member

macho-catt commented Nov 3, 2021

Two new issues need to be created so this can be closed:

  1. Issue to update no-labels-template.md so that the message given by the bot is up to date with how the current issue creation workflow is the message is already updated so this is done
  2. Issue to update add-labels-tempate.md so that the bot creates a message if labels are missing.

@macho-catt macho-catt self-assigned this Nov 16, 2021
@github-actions

This comment has been minimized.

@github-actions github-actions bot added To Update ! No update has been provided and removed To Update ! No update has been provided labels Nov 22, 2021
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions github-actions bot added To Update ! No update has been provided and removed To Update ! No update has been provided labels Dec 3, 2021
@github-actions
Copy link

@macho-catt

Please add update using the below template (even if you have a pull request). Afterwards, remove the 'To Update !' label and add the 'Status: Updated' label.

  1. Progress: "What is the current status of your project? What have you completed and what is left to do?"
  2. Blockers: "Difficulties or errors encountered."
  3. Availability: "How much time will you have this week to work on this issue?"
  4. ETA: "When do you expect this issue to be completed?"
  5. Pictures: "Add any pictures of the visual changes made to the site so far."

If you need help, be sure to either: 1) place your issue in the developer meeting discussion column and ask for help at your next meeting, 2) put a "Status: Help Wanted" label on your issue and pull request, or 3) put up a request for assistance on the #hfla-site channel.

You are receiving this comment because your last comment was before Monday, December 6, 2021 at 11:19 PM PST.

@github-actions github-actions bot added the To Update ! No update has been provided label Dec 10, 2021
@macho-catt
Copy link
Member

Will finish this issue this week (by Sunday)
Availability: 2 hours

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Small Take this type of issues after the successful merge of your second good first issue Feature: Board/GitHub Maintenance Project board maintenance that we have to do repeatedly role: back end/devOps Tasks for back-end developers size: missing Status: Updated No blockers and update is ready for review
Projects
Development

Successfully merging a pull request may close this issue.

4 participants