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

Added parameters to override base/head commits #314

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dkhunt27
Copy link

PR Checklist

Please check if your PR fulfills the following requirements:

- [x] The commit message follows our guidelines
- [x Tests for the changes have been added (if applicable)
- [x] Docs have been added / updated (if applicable)

PR Type

What kind of change does this PR introduce?

- [ ] Bugfix
- [x] Feature
- [ ] Code style update (formatting, local variables)
- [ ] Refactoring (no functional changes, no api changes)
- [ ] Build related changes
- [ ] CI related changes
- [ ] Other... Please describe:

What is the current behavior

Added new parameters to override the head and base commits when running nx commands . This will allow user to override pipeline if they need to go back more than one commit or to a specific SHA

Issue Number fixed #n/a

What is the new behavior

There are new inputs baseBoundaryOverride and headBoundaryOverride which when specified will override the base/head commit for the nx command. You can use it like so to get all commits since last release/tag

Does this PR introduce a breaking change

- [ ] Yes
- [x] No

Other information

@sonarcloud
Copy link

sonarcloud bot commented Oct 27, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@jonsoku-dev
Copy link

i want to merge this pr....
i need it..!

@yusijs
Copy link

yusijs commented May 22, 2023

Would love this as well; we build some of our environments outside of pull requests, so we need to be able to tell it to target main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants