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

USWDS-Team - POAM: Use dynamic date in issue creation #457

Merged
merged 3 commits into from
Jan 3, 2025

Conversation

mahoneycm
Copy link
Contributor

@mahoneycm mahoneycm commented Dec 20, 2024

Summary

Updated POAM GitHub workflow to dynamically set the month and year and add the triage label. Now, team members will no longer have to manually update to match the current date.

Related issue

Closes #416

Problem statement

Automatic POAM issue creation lacks meta data and includes hard coded values

Solution

  • Use dynamic value setting to automatically set the month and year of the issue summary and title.
  • Add triage label to label list

Testing and review

  1. Visit test issue and confirm the description matches expectations
  2. Review test workflow and confirm it runs as expected

Note

Test workflow also attempts the add the issue to our project board. This is currently not possible and may be resolved via #458

Important

After merge, we should delete the cm-POAM-test-workflow branch to remove the test workflow from our actions page

Additionally, reorder repo list to closer match priority
@mahoneycm mahoneycm marked this pull request as ready for review December 20, 2024 21:31
@mahoneycm mahoneycm requested a review from a team as a code owner December 20, 2024 21:31
mejiaj
mejiaj previously approved these changes Dec 20, 2024
@@ -11,27 +11,30 @@ jobs:
permissions:
issues: write
steps:
- name: Dynamically set DATE environment variable
run: echo "DATE=$(date '+%B %Y')" >> $GITHUB_ENV
Copy link
Contributor

Choose a reason for hiding this comment

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

Tip

Outputs December 2024

.github/workflows/POAM.yml Outdated Show resolved Hide resolved
@mahoneycm mahoneycm requested a review from mejiaj December 23, 2024 17:35
Copy link
Contributor

@amyleadem amyleadem left a comment

Choose a reason for hiding this comment

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

LGTM!

@thisisdano thisisdano merged commit 65b6918 into main Jan 3, 2025
@thisisdano thisisdano deleted the cm-POAM-workflow-dynamic-date branch January 3, 2025 20:44
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.

USWDS-Team - Enhancement: POAM automation enhancements
4 participants