matous-volf/todo-baggins
1
0
Fork 0
Code Issues 26 Pull Requests Actions Projects 1 Activity

ci: GitHub action for code linting #6

Merged
matous-volf merged 5 commits from ci/lint-check into main 2024-08-15 11:55:20 +00:00
Conversation 5 Commits 5 Files Changed 3 +5 -3
2 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e130da77f4 - Show all commits

6
.github/workflows/conventional-commits.yml vendored
View File

@@ -9,7 +9,9 @@ jobs:
name: conventional commits
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v3
- uses: webiny/action-conventional-commits@v1.3.0
- name: checkout code
uses: actions/checkout@v3
- name: conventional commits check
uses: webiny/action-conventional-commits@v1.3.0
with:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

2
.github/workflows/conventional-pull-requests.yml vendored
View File

@@ -8,7 +8,7 @@ jobs:
validate-pr-title:
coderabbitai[bot] commented 2024-08-15 11:53:17 +00:00 (Migrated from github.com)
Review
Copy Link

Define explicit permissions for security.

The addition of an empty permissions block at the top level is unnecessary since specific permissions are already defined within the build job. Consider removing it to avoid confusion.

- permissions: { }
Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
**Define explicit permissions for security.** The addition of an empty `permissions` block at the top level is unnecessary since specific permissions are already defined within the `build` job. Consider removing it to avoid confusion. ```diff - permissions: { } ``` <!-- suggestion_start --> <details> <summary>Committable suggestion</summary> > :bangbang: **IMPORTANT** > Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements. `````suggestion ````` </details> <!-- suggestion_end --> <!-- This is an auto-generated comment by CodeRabbit -->
runs-on: ubuntu-22.04
steps:
- name: PR Conventional Commit Validation
- name: conventional pull requests check
uses: ytanikin/PRConventionalCommits@1.1.0
with:
task_types: '["build","chore","ci","docs","feat","fix","perf","refactor","revert","style","test"]'