department-of-veterans-affairs / notification-api

Notification API
MIT License
16 stars 9 forks source link

Simplify PR labeling that versions deploys #1970

Closed coreycarvalho closed 1 month ago

coreycarvalho commented 2 months ago

User Story - Business Need

Our CI/CD pipeline uses PR labels to determine how to bump the version of our code. We use semver versioning, which contains 3 parts (major.minor.patch), but we use more than 3 labels that eventually map to the 3 part semver scheme. This is cumbersome for developers.

We'd like to simplify this to use only 3 labels: major, minor, and patch. Those labels will match 1:1 to the semver versioning scheme.

User Story(ies)

As a VA Notify developer I want to simplify our PR labeling scheme to align with semver So that I can have a more streamlined development process.

Additional Info and Resources

Acceptance Criteria

QA Considerations

Potential Dependencies

npmartin-oddball commented 1 month ago

Hey team! Please add your planning poker estimate with Zenhub @coreycarvalho @cris-oddball @EvanParish @k-macmillan @kalbfled @MackHalliday @mchlwellman

coreycarvalho commented 1 month ago

Updated https://docs.google.com/document/d/1nwZIF_lydPWfvixxZlQLNt4nqy3Qp13pHQnMcYJjTqE/edit with the new labels

cris-oddball commented 1 month ago

Ticket is just here to remind me to send the two commits up through the pipeline for

cris-oddball commented 1 month ago

Since we are not rolling back the API, these commits will automatically go up to prod on the next API commit for the IIR fix. No need to hold open this ticket.