Skip to content

Allow to set pull request custom title and body #137

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

Merged
merged 1 commit into from
Jul 5, 2023

Conversation

eko
Copy link
Contributor

@eko eko commented Jun 29, 2023

Hi,

We are using your GitOps rules and would like to add some arguments to the @com_adobe_rules_gitops//gitops/prer:create_gitops_prs in order to allow settings a custom title and/or body pull request.

Description

By default, it continues to act as it is for now by settings the following as title and body: GitOps deployment <branch>, but you can also specify:

  • -gitops_pr_title "<your pull request title>"
  • -gitops_pr_body "<your pull request body>"

This would be useful for us in order to display a changelog in the pull request body.

Related Issue

#136

Motivation and Context

We would like to add custom information to the pull request such as a changelog.

How Has This Been Tested?

I've tested it by forking the project and using it in my Bazel project.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@eko eko requested a review from a team as a code owner June 29, 2023 06:56
Copy link
Collaborator

@michaelschiff michaelschiff 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 the contribution! LGTM

@eko eko changed the title Allow to set pull request body when defined Allow to set pull request custom title and body Jun 29, 2023
@eko
Copy link
Contributor Author

eko commented Jul 4, 2023

Hello, do you think we could have this merged and released soon?

@michaelschiff michaelschiff merged commit 64c0dc5 into adobe:main Jul 5, 2023
@michaelschiff
Copy link
Collaborator

Sorry for the delay, thank you again for the contribution!

@eko
Copy link
Contributor Author

eko commented Jul 5, 2023

No problem, thank you @michaelschiff! :)

@eko
Copy link
Contributor Author

eko commented Jul 5, 2023

@michaelschiff Could you also please tag a new release with this addition?

arturo-skydio pushed a commit to Skydio/rules_gitops that referenced this pull request Oct 5, 2023
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