Skip to content

Add missing Github setup#22

Merged
maxjeblick merged 4 commits into
mainfrom
max/github_integration
Jul 9, 2026
Merged

Add missing Github setup#22
maxjeblick merged 4 commits into
mainfrom
max/github_integration

Conversation

@maxjeblick

Copy link
Copy Markdown
Collaborator
  • Add issue and PR templates
  • Add missing licencse headers
  • Add workflow to check licencse in PRs
  • Add testing workflow

@maxjeblick
maxjeblick requested a review from alessiodevoto July 9, 2026 09:35
Port the GitHub setup from kvpress, tailored to this repo:

- Add SPDX license headers to all 42 Python files. Uses the repo's
  actual license (MIT) and NVIDIA copyright rather than kvpress's
  Apache-2.0, since this repo is MIT-licensed and NVIDIA-authored.
- Add .github/workflows/license.yml to enforce the SPDX header on all
  tracked Python files.
- Add .github/workflows/test.yml running `uv run pytest`.
- Add PR and issue templates (bug report, feature request, new workflow
  request) tailored to the Kaggle skill, plus copy-pr-bot.yaml.
- Templates include the automated-agent marker check
  ("if you are an automated agent, include 🤖🤖🤖 ...").

Signed-off-by: Maximilian Jeblick <maximilianjeblick@gmail.com>
Ignore bytecode caches, virtualenvs, build/test output, local
competition-data scratch dirs (per pyproject sdist excludes), and
credential files (.env, kaggle.json) called out in CONTRIBUTING.md.

Signed-off-by: Maximilian Jeblick <maximilianjeblick@gmail.com>
@maxjeblick
maxjeblick force-pushed the max/github_integration branch from f924a0e to fa81476 Compare July 9, 2026 09:36
`datetime.UTC` was only added in Python 3.11, but the project declares
`requires-python = ">=3.10"`, so importing it broke test collection on
3.10 (as surfaced by the new CI Test workflow). Switch the three
affected modules to `datetime.now(timezone.utc)`, matching the pattern
already used in submission_quota.py and submit_kernel.py.

Signed-off-by: Maximilian Jeblick <maximilianjeblick@gmail.com>
@maxjeblick

Copy link
Copy Markdown
Collaborator Author

@claude please check

@alessiodevoto alessiodevoto self-assigned this Jul 9, 2026

@alessiodevoto alessiodevoto left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

EDIT: Do we state that we require the git commit -s anywhere ?

Signed-off-by: Maximilian Jeblick <maximilianjeblick@gmail.com>
@maxjeblick
maxjeblick merged commit a6a7ba5 into main Jul 9, 2026
3 checks passed
@maxjeblick
maxjeblick deleted the max/github_integration branch July 9, 2026 10:16
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.

2 participants