Skip to content

feat: Add validation step that runs on approved PR review - #50

Open
nathaniel-itential wants to merge 2 commits into
itential:mainfrom
nathaniel-itential:feat/add-validation-step
Open

feat: Add validation step that runs on approved PR review#50
nathaniel-itential wants to merge 2 commits into
itential:mainfrom
nathaniel-itential:feat/add-validation-step

Conversation

@nathaniel-itential

Copy link
Copy Markdown
Contributor

Add a validation step, runs on approved PR review. Adjusted diff logic to accept BASE_SHA parameter instead of hardcoding HEAD~1 commit for diff logic.

Validation.yml does the diff and artifact setup for the validation-deploy step, and runs on review approval. Validation-deploy.yml runs when validation.yml is finished, and deploys the assets to staging.

@nathaniel-itential

Copy link
Copy Markdown
Contributor Author

@jennlu330 Adjusted the diff script and validation.yml script to take in a env variable called INCLUDE_ALL_SPEC_VERSIONS that decides to collect all specs or just latest specs. Not defining the variable defaults to only collecting -latest, so promotion.yml does not need to be updated.

@42nacosta 42nacosta left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Test

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.

2 participants