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.

Sign up for GitHub

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

Jump to bottom

Fix commit message options -- Rework Update/Merge Logic #4

Merged
merged 11 commits into from
Mar 16, 2024
Merged

Fix commit message options -- Rework Update/Merge Logic #4

merged 11 commits into from
Mar 16, 2024

Conversation

F-WRunTime
Copy link
Member

@F-WRunTime F-WRunTime commented Mar 15, 2024 •
edited
Loading

Purpose:

  • Developers do not always want comments predefined for them for the squashed commit from the PR to target branch
  • PRs were being updated when test are still in progress or failing & actively being developed on

Solutions / Implementation / changes

  • Rework / overhaul of logic to determine if a PR is up to date / passing, up-to-date / failing, overall failing/pending
  • Adding logic to enable the use of supplied comments, and defaults to assuming user wants their PR Body to be used as the commit message.

devops and others added 7 commits February 13, 2024 15:11
Update workflow to use debug mode by choice, not required. Update act…
e91699b 
…ion to include use of dry-run. Change test Yaml to point to special repository for testing
Test workflow now with dry-run as an option from the workflow action
0073958
remove basic validation, add test to pull request pass requirements
718b183
pass flag --dry-run instead of bool
aad8f48
Updating instructions. Fixing usage of Github function merge()
bdce0cd
Update README about test execution and purpose. Try and set output fr…
8bc67a4 
…om action to enable validation of test results when action completes with a results output. Update automerge to cleanly handle buckets of PRs to pass/updated, pass/outofdate, and failed/pending scenarios
@F-WRunTime
Merge branch 'master' into fix/rewrite-commit-message
1905bfa
@F-WRunTime F-WRunTime changed the title Fix/rewrite commit message Fix commit message options -- Rework Update/Merge Logic Mar 15, 2024
@F-WRunTime F-WRunTime merged commit 915a1bb into master Mar 16, 2024
2 checks passed
@F-WRunTime F-WRunTime deleted the fix/rewrite-commit-message branch March 16, 2024 04:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@F-WRunTime