Skip to content

ci: add validation guard to generate-keys workflow#1119

Merged
saurabhraghuvanshii merged 3 commits into
layer5io:masterfrom
rishiraj38:fix/generate-keys-workflow-guard
Jun 24, 2026
Merged

ci: add validation guard to generate-keys workflow#1119
saurabhraghuvanshii merged 3 commits into
layer5io:masterfrom
rishiraj38:fix/generate-keys-workflow-guard

Conversation

@rishiraj38

Copy link
Copy Markdown
Member

Description

This PR adds a validation guard to the daily generate-keys.yml workflow. This ensures that any downloaded permissions spreadsheet file is valid and correctly formatted before it gets automatically committed to the repository, preventing corrupted files or Google Sheets error pages from breaking the site.

Changes

  • Updated the download step to use the curl -f flag to fail immediately on HTTP error status codes.
  • Added a validation check using grep to ensure keys.csv contains the expected column headers (Category,Function,Feature).
  • Exits early with exit 1 if validation fails, preventing corrupted files from being committed.

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@welcome

welcome Bot commented Jun 23, 2026

Copy link
Copy Markdown

Yay, your first pull request! 👍 A contributor will be by to give feedback soon. In the meantime, please review the Layer5 Community Welcome Guide and sure to join the community Slack.
Be sure to double-check that you have signed your commits. Here are instructions for making signing an implicit activity while peforming a commit.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@github-actions

github-actions Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.6.3
Preview removed because the pull request was closed.
2026-06-24 18:18 UTC

@saurabhraghuvanshii saurabhraghuvanshii merged commit ffd11a9 into layer5io:master Jun 24, 2026
4 checks passed
@welcome

welcome Bot commented Jun 24, 2026

Copy link
Copy Markdown

        Thank you for contributing to the Layer5 community! 🎉 \ \ Congrats! \ \         ⭐ Please leave a star on the project. 😄

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants