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

Cleanup release create around new --notes-from-tag flag #8016

Merged
merged 2 commits into from Sep 19, 2023

Conversation

samcoe
Copy link
Contributor

@samcoe samcoe commented Sep 18, 2023

This PR is a follow up to #7861. It addresses some of the documentation, and testing concerns I had. It also simplifies the implementation by removing an unnecessary API call. As of right now the --notes-from-tag feature does not work in conjunction the --repo flag but we should address that in the near future.

@samcoe samcoe self-assigned this Sep 18, 2023
@samcoe samcoe marked this pull request as ready for review September 18, 2023 12:50
@samcoe samcoe requested a review from a team as a code owner September 18, 2023 12:50
@samcoe samcoe requested review from williammartin and removed request for a team September 18, 2023 12:50
@cliAutomation cliAutomation added this to Needs review 🤔 in The GitHub CLI Sep 18, 2023
@samcoe samcoe enabled auto-merge (squash) September 19, 2023 08:19
Copy link
Member

@williammartin williammartin left a comment

Choose a reason for hiding this comment

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

LGTM thanks for the tests especially.

@samcoe samcoe merged commit 94fbbdf into trunk Sep 19, 2023
10 checks passed
@samcoe samcoe deleted the cleanup-release-create branch September 19, 2023 08:26
dmgardiner25 pushed a commit to dmgardiner25/cli that referenced this pull request Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
The GitHub CLI
  
Needs review 🤔
Development

Successfully merging this pull request may close these issues.

None yet

3 participants