Skip to content

ci: block new Python files via PR check#5600

Merged
marsninja merged 16 commits intojaseci-labs:mainfrom
SandeepaHWP:fix/no-python
Apr 20, 2026
Merged

ci: block new Python files via PR check#5600
marsninja merged 16 commits intojaseci-labs:mainfrom
SandeepaHWP:fix/no-python

Conversation

@SandeepaHWP
Copy link
Copy Markdown
Collaborator

@SandeepaHWP SandeepaHWP commented Apr 17, 2026

Summary

Adds a GitHub Actions workflow that fails any PR introducing new .py files.

What it does:

  • Runs on every pull request (open, update, reopen, or label added)
  • Checks if any new .py files are being added (--diff-filter=A catches additions only)
  • Fails with a clear error message if any are found

To skip this check:
Add the skip-no-python-check label to your PR. The check will re-run automatically when the label is added and will pass.

And 3 quick parallel jobs were merged into one for better readability of checks.

@SandeepaHWP SandeepaHWP changed the title (Check): No Python Files ci: block new Python files via PR check Apr 18, 2026
@kugesan1105
Copy link
Copy Markdown
Member

add a label to ignore this check!
also move this to a part of some existing Jobs

@kugesan1105
Copy link
Copy Markdown
Member

combine them to single sub jobs
image

Copy link
Copy Markdown
Member

@kugesan1105 kugesan1105 left a comment

Choose a reason for hiding this comment

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

LGTM

@marsninja marsninja enabled auto-merge (squash) April 20, 2026 18:35
@marsninja marsninja merged commit ea9b714 into jaseci-labs:main Apr 20, 2026
11 checks passed
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.

3 participants