Skip to content

Ci triggers matrix #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ameertechy wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Ci triggers matrix #46

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
b41c094
Add CI workflow for linting and Docker build
ameertechy Jun 20, 2026
1679793
Remove code-lint job from CI workflow
ameertechy Jun 20, 2026
00b966c
Merge branch 'master' into master
ameertechy Jun 21, 2026
1a93bd0
Merge branch 'LondheShubham153:master' into master
ameertechy Jun 21, 2026
b98369c
Merge branch 'LondheShubham153:master' into master
ameertechy Jun 28, 2026
adf3496
Add PR trigger workflow
ameertechy Jul 6, 2026
dce5315
Rename pr-check.yml workflow file
ameertechy Jul 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 0 additions & 43 deletions .github/workflows/code-quality.yml
View file
Open in desktop

This file was deleted.

38 changes: 0 additions & 38 deletions .github/workflows/code-tests.yml
View file
Open in desktop

This file was deleted.

24 changes: 0 additions & 24 deletions .github/workflows/dast.yml
View file
Open in desktop

This file was deleted.

47 changes: 0 additions & 47 deletions .github/workflows/dependency-scan.yml
View file
Open in desktop

This file was deleted.

28 changes: 0 additions & 28 deletions .github/workflows/deploy.yml
View file
Open in desktop

This file was deleted.

53 changes: 0 additions & 53 deletions .github/workflows/devsecops.yml
View file
Open in desktop

This file was deleted.

31 changes: 0 additions & 31 deletions .github/workflows/docker-push.yml
View file
Open in desktop

This file was deleted.

44 changes: 0 additions & 44 deletions .github/workflows/docker-scans.yml
View file
Open in desktop

This file was deleted.

33 changes: 0 additions & 33 deletions .github/workflows/matrix.yml
View file
Open in desktop

This file was deleted.

14 changes: 14 additions & 0 deletions .github/workflows/pr-check.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
name: PR Check

on:
pull_request:
branches: [master]

jobs:
check:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Print PR branch
run: echo "PR check running for branch: ${{ github.head_ref }}"
Comment on lines +12 to +13

@coderabbitai coderabbitai Bot Jul 6, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 Functional Correctness | 🔴 Critical | ⚡ Quick win

YAML syntax error breaks the workflow.

The unquoted run value contains branch: (colon+space), which YAML parses as a nested mapping key rather than plain text. This is confirmed by both yamllint and actionlint reporting a parse failure at line 13 — the workflow will fail to load and the job will never execute.

🐛 Proposed fix: quote the run string 
       - name: Print PR branch
-        run: echo "PR check running for branch: ${{ github.head_ref }}"
+        run: echo "PR check running for branch: ${{ github.head_ref }}"

Actually wrap the whole value in single quotes so the embedded colon is safe:

       - name: Print PR branch
-        run: echo "PR check running for branch: ${{ github.head_ref }}"
+        run: 'echo "PR check running for branch: ${{ github.head_ref }}"'
📝 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.

Suggested change
- name: Print PR branch
run: echo "PR check running for branch: ${{ github.head_ref }}"
- name: Print PR branch
run: 'echo "PR check running for branch: ${{ github.head_ref }}"'
🧰 Tools
🪛 actionlint (1.7.12)

[error] 13-13: could not parse as YAML: mapping values are not allowed in this context

(syntax-check)

🪛 YAMLlint (1.37.1)

[error] 13-13: syntax error: mapping values are not allowed here

(syntax)

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.github/workflows/pr-check.yml around lines 12 - 13, The workflow step in
pr-check.yml has a YAML parsing issue because the run value in the Print PR
branch step contains an unquoted colon+space, so quote the entire echo command
string to make it valid YAML. Update the run content in the Print PR branch step
so the shell command is treated as plain text and the workflow can load
successfully. Refer to the Print PR branch step and its run field when making
the change.

Source: Linters/SAST tools


20 changes: 0 additions & 20 deletions .github/workflows/secret-scanning.yml
View file
Open in desktop

This file was deleted.

Loading