Skip to content

[CERT-210] - GitHub Actions: lint fixed#57

Closed
josephcampos-gap wants to merge 6 commits into
mainfrom
github-actions-lint-fix
Closed

[CERT-210] - GitHub Actions: lint fixed#57
josephcampos-gap wants to merge 6 commits into
mainfrom
github-actions-lint-fix

Conversation

@josephcampos-gap

Copy link
Copy Markdown
Contributor

This pull request introduces several improvements and updates across the Bruno lint workflow and its associated validation files. The main focus is on making the workflow easier to test locally, updating validation files to use placeholder values for better reusability, and refining linting rules for more accurate error detection.

Documentation and Workflow Improvements:

  • Added a comprehensive guide .github/TESTING_ACTIONS_LOCALLY.md detailing how to test GitHub Actions locally using act, including troubleshooting, direct linter execution, and workflow success criteria.
  • Updated workflow trigger paths in .github/workflows/lint-bruno.yml to reference files within the bruno directory, ensuring the workflow only runs when relevant files change.

Validation File Updates (Bruno .bru files):

  • Refactored multiple .bru validation files to replace hardcoded values in URLs and query parameters with [ENTER_*] placeholders, making them more generic and easier to configure for different environments. This affects files under StaffAssociation, StudentGrade, and StudentTranscript test suites. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]

Linting Rule Refinements:

  • Changed the Bruno linter rule for hardcoded descriptor URIs in validation files from a warning to an error, enforcing stricter validation.
  • Updated the linter to accept both encodeUrl: true and encodeUrl: false in settings blocks, and improved the patch logic for missing encodeUrl settings.

@github-actions

Copy link
Copy Markdown

Bruno Lint Report

Files Scanned: 0
Problems: 0 (Errors: 0, Warnings: 0)
✅ No blocking errors.

Triggered by workflow: lint-bruno.yml

@github-actions

Copy link
Copy Markdown

Bruno Lint Report

Files Scanned: 278
Problems: 0 (Errors: 0, Warnings: 0)
✅ No blocking errors.

Triggered by workflow: lint-bruno.yml

@josephcampos-gap josephcampos-gap changed the title [GitHub Actions] - lint fixed [CERT-210] - GitHub Actions: lint fixed Oct 30, 2025
@github-actions

Copy link
Copy Markdown

Bruno Lint Report

Files Scanned: 278
Problems: 0 (Errors: 0, Warnings: 0)
✅ No blocking errors.

Triggered by workflow: lint-bruno.yml

@github-actions

Copy link
Copy Markdown

Bruno Lint Report

Files Scanned: 302
Problems: 0 (Errors: 0, Warnings: 0)
✅ No blocking errors.

Triggered by workflow: lint-bruno.yml

@josephcampos-gap

Copy link
Copy Markdown
Contributor Author

I'm closing it since it's already part of PR #58

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.

1 participant