Skip to content

feat: workflows#1

Merged
Adames4 merged 4 commits intomasterfrom
feature/workflows
Feb 6, 2026
Merged

feat: workflows#1
Adames4 merged 4 commits intomasterfrom
feature/workflows

Conversation

@Adames4
Copy link
Member

@Adames4 Adames4 commented Feb 5, 2026

Summary by CodeRabbit

  • Chores
    • Added a reusable GitHub Actions workflow for Python linting (triggers: pushes, PRs, manual dispatch).
    • Removed the CI lint stage from the pipeline configuration.
    • Updated project metadata (name renamed, author added) and tightened Python version requirement.
    • Updated and reorganized dependencies, added dependency groups and external sources.
    • Removed local script shortcuts previously provided for common tasks.

@gemini-code-assist
Copy link

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026

Warning

Rate limit exceeded

@Adames4 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 1 minutes and 44 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📝 Walkthrough

Walkthrough

Adds a wrapper GitHub Actions workflow for reusable Python linting, removes a GitLab CI lint include/stage, and updates project metadata, dependencies, dependency groups, and source configuration in pyproject.toml (including removal of pdm scripts and tools sections).

Changes

Cohort / File(s) Summary
GitHub Actions workflow
.github/workflows/python-lint.yml
New workflow that delegates to a reusable lint workflow (RationAI/.github/.github/workflows/python-lint.yml@main); triggers: push to master, PRs to master, workflow_dispatch.
GitLab CI
.gitlab-ci.yml
Removed include of external Python-Lint template and removed the lint stage/job from the pipeline.
Project config
pyproject.toml
Major metadata and dependency updates: project name, author, Python requirement tightened; dependencies changed/updated; dev/run dependency groups redefined; removed [tool.pdm.scripts] and [tool.pdm]; added [tool.uv.sources] with two git sources (rationai-mlkit, rationai-kube-jobs).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 I hopped through CI, a tiny chore,
Wrapped lint in a workflow, then asked for more,
Removed an old stage, tuned deps with care,
New sources and names now dance in the air,
Sniff, sniff — the tree of code smells lighter I swear! 🥕✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'feat: workflows' is overly vague and generic, failing to clearly summarize the actual changes which include GitHub Actions workflow addition, GitLab CI removal, and significant pyproject.toml restructuring. Provide a more specific title that captures the primary changes, such as 'feat: migrate CI workflows from GitLab to GitHub Actions' or 'feat: update project dependencies and migrate CI configuration'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/workflows

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In @.github/workflows/python-lint.yml:
- Around line 10-11: Replace the floating reusable workflow reference "uses:
RationAI/.github/.github/workflows/python-lint.yml@main" with an immutable ref
(a full commit SHA or a release tag) so upstream changes cannot silently alter
CI; update the uses line in the workflow to target the chosen commit SHA or tag
string (the existing uses declaration is the unique identifier to change), and
include a short comment in the file noting the chosen SHA/tag and why it was
pinned.

@Adames4 Adames4 self-assigned this Feb 5, 2026
@Adames4 Adames4 requested a review from matejpekar February 5, 2026 16:20
@Adames4 Adames4 requested a review from matejpekar February 5, 2026 17:01
@Adames4 Adames4 merged commit 2cd9e65 into master Feb 6, 2026
2 of 3 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.

2 participants