admin: migrate to centralized linting workflows#116
Merged
Conversation
Applied via project-admin workflow Repository: OTPValidation Operation: centralize-linting-workflows
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports |
Contributor
Collaborator
|
@hdamker sorry I'm bit lost - we can merge this one, right? |
Contributor
@bigludo7 you are right, there is currently no requirement to pass the check before a PR can be merged. I'm currently investigation if (and if yes, why) this was the case with the legacy linting. For now it might be an advantages that the check is non-blocking. But I suppose that we will require from Release Management that there are no linting errors left before the public release. |
bigludo7
approved these changes
Aug 21, 2025
Collaborator
|
@hdamker - @fernandopradocabrillo - I've fixed all linting issues - I proposed to merge this one :) |
fernandopradocabrillo
approved these changes
Aug 22, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CAMARA Project Admin Update - Linting Migration
This pull request migrates this repository from local linting configuration to centralized linting workflows managed by the CAMARA project.
🔄 Migration Summary
Removed local linting artifacts:
megalinter.ymlspectral_oas_lint.yml.spectral.yml.yamllint.yaml/lint_function/Added centralized workflows:
spectral-oas-caller.yml- Spectral linting with CAMARA rulesetpr_validation_caller.yml- Comprehensive PR validation✨ Benefits of Centralized Linting
📋 What This Means for You
🔧 Technical Details
The new workflows reference reusable workflows from:
camaraproject/tooling/.github/workflows/This ensures all repositories benefit from:
👥 Next Steps for Codeowners
Before This PR Can Be Merged:
Review linting results in PR checks:
Fix linting errors directly in this PR:
Approve and merge this PR 🚀
After Successful Merge:
Monitor the new linting system:
Test with additional rules (optional):
Monitor future PRs:
💡Pro tip: Running the Spectral workflow manually NOW is highly recommended. This allows you to fix issues proactively rather than discovering them when submitting your next feature PR!
🤖 Generated via project-admin workflow
Triggered by hdamker, executed via hdamker-bot
➡️ Next Steps: This PR should be reviewed, fixed as needed, approved, and merged by repository codeowners following standard review processes.
This is a manually triggered automated administrative update.