admin: migrate to centralized linting workflows#388
Conversation
Applied via project-admin workflow Repository: EdgeCloud Operation: centralize-linting-workflows
🦙 MegaLinter status: ❌ ERROR
See detailed report in MegaLinter reports |
|
@camaraproject/edge-cloud_codeowners The centralized linting includes also the test definition files, hence the failed check. It might make sense to do the clean-up and proper naming of the API and test definition files first, or even better to move the remaining API definitions with their test definition also in into own dedicated repositories. |
|
Thanks @hdamker , we are already trying to clean up the repository, removing and reordering the different files, we have some PRs open related with this topic. |
@JoseMConde My view is that it was the much cleaner solution to split also the last API out into a new repository and keep the EdgeCloud repository either only for documentation across the EdgeCloud related APIs or even archive it completely at some point of time. |
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.gherkin-lintrc/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
Note on Linting Checks: The linting workflows are currently not blocking - they will not prevent this PR (or future PRs) from being merged. However, it is highly recommended to:
This approach maintains code quality while allowing flexibility during the transition period.
Before Merging This PR:
Review linting results in PR checks:
Fix linting errors (strongly recommended):
Approve and merge this PR 🚀
After Merge:
Establish linting best practices:
Test with additional rules (optional):
Monitor future PRs:
💡Pro tip: Running the Spectral workflow manually NOW is highly recommended. This allows you to identify and 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.