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

chore(extras): script to generate list of PRs that master is ahead of rc #701

Merged
merged 1 commit into from
Jul 12, 2023

Conversation

jansegre
Copy link
Member

Motivation

Getting the list of PRs to include in the release-candidate description was getting cumbersome and error prone.

Acceptance Criteria

  • new script in extras that will generate a markdown list of PRs to be used when creating a new release-candidate
  • script requires gh to be installed and authenticated in order to use Github's API

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

@jansegre jansegre self-assigned this Jul 12, 2023
@jansegre jansegre requested a review from msbrogli as a code owner July 12, 2023 19:47
@jansegre jansegre requested a review from luislhl July 12, 2023 19:48
msbrogli
msbrogli previously approved these changes Jul 12, 2023
@@ -0,0 +1,64 @@
#!/bin/env python
Copy link
Member

Choose a reason for hiding this comment

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

Add some docs on how to execute and give an example of output.

Copy link
Member Author

Choose a reason for hiding this comment

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

Done.

@codecov
Copy link

codecov bot commented Jul 12, 2023

Codecov Report

Merging #701 (f3a9ce6) into master (acb0972) will increase coverage by 0.01%.
The diff coverage is n/a.

❗ Current head f3a9ce6 differs from pull request most recent head 32ae482. Consider uploading reports for the commit 32ae482 to get more accurate results

@@            Coverage Diff             @@
##           master     #701      +/-   ##
==========================================
+ Coverage   84.11%   84.12%   +0.01%     
==========================================
  Files         246      246              
  Lines       20448    20450       +2     
  Branches     2760     2760              
==========================================
+ Hits        17200    17204       +4     
+ Misses       2674     2673       -1     
+ Partials      574      573       -1     

see 5 files with indirect coverage changes

@@ -0,0 +1,79 @@
#!/usr/bin/env python
Copy link
Collaborator

Choose a reason for hiding this comment

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

We need a way for people doing releases to know that this exists (in case someone else needs to do them)

I think we should guide them to use this in our release guide.
What do you think?

Copy link
Member Author

Choose a reason for hiding this comment

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

That would be great.

@jansegre jansegre force-pushed the chore/gen-rc-prs-script branch from f3a9ce6 to 32ae482 Compare July 12, 2023 23:10
@jansegre jansegre merged commit d920df5 into master Jul 12, 2023
@jansegre jansegre deleted the chore/gen-rc-prs-script branch July 12, 2023 23:20
@jansegre jansegre mentioned this pull request Jul 24, 2023
2 tasks
This was referenced Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants