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

[Enabler][692] Add a command that checks the changelog into ac and create a GitHub action #1422

Open
wants to merge 16 commits into
base: dev
Choose a base branch
from

Conversation

IsaacVRey
Copy link
Contributor

@IsaacVRey IsaacVRey commented Apr 5, 2024

SUMMARY

This PR fixes the first two bullet points of #692.

Add new command ac-check-changelog into ac tool to ensure if a new changelog fragment was added to the branch

Added new github action that will execute the check changelog check upon a new pull request with base branch dev or any branch that starts with staging. By default, it will run on three triggers:

  • open
  • synchronize (every time the branch, with its PR in draft mode or ready for review gets a new push into remote).
  • reopen

You can see this Github Action in the following PR fernandofloresg#24
And look at the run details here https://github.com/fernandofloresg/ibm_zos_core/actions/runs/8574374819/job/23501055062?pr=24

It averages 12s of execution time.
image

And looking at billable details we get 0s,
image

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

ac

ADDITIONAL INFORMATION

@IsaacVRey IsaacVRey marked this pull request as ready for review April 5, 2024 20:17
@fernandofloresg fernandofloresg added the Do not Merge When a pull request should not be merged for issue noted reasons label Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do not Merge When a pull request should not be merged for issue noted reasons
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants