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

Add release drafter github action #1604

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

chiaberry
Copy link
Member

@chiaberry chiaberry commented Nov 15, 2024

Associated issues

cityofaustin/atd-data-tech#19588

Testing

https://github.com/marketplace/actions/release-drafter

I think the only way to test is see that its working as PRs are merged to main.

There are many ways to configure the drafter, some that might be useful for VZ

  • specify which paths to include in release notes - include-paths: (i wish we could specify what to exclude, like the app folder but its a whitelist type thing)
  • add headers to the template, like Editor, Viewer, API etc
  • and if you feel like reworking how the changes are formatted, you can change-template: '- $TITLE @$AUTHOR (#$NUMBER)'

Ship list

  • Check migrations for any conflicts with latest migrations in main branch
  • Confirm Hasura role permissions for necessary access
  • Code reviewed
  • Product manager approved

Copy link
Member

@johnclary johnclary left a comment

Choose a reason for hiding this comment

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

Thanks for setting this up! I feel good about rolling with the default settings for now 🚢 🚢 🚢 🚢

Copy link
Contributor

@mddilley mddilley left a comment

Choose a reason for hiding this comment

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

Approving since this action worked beautifully when you merged in the Moped repo. Thanks for patching these up!

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.

3 participants