Skip to content

Update python and pre-commit versions#50

Open
NKeleher wants to merge 1 commit intomainfrom
chore/update-reqs-2026-02-02
Open

Update python and pre-commit versions#50
NKeleher wants to merge 1 commit intomainfrom
chore/update-reqs-2026-02-02

Conversation

@NKeleher
Copy link
Contributor

@NKeleher NKeleher commented Feb 2, 2026

Pull Request Summary 🚀

What does this PR do? 📝

Bundling python package updates.

Why is this change needed? 🤔

Updates to python packages, especially nbstata and jupyterlab for improved functionality

How was this implemented? 🛠️

ran uv add {package}>={version} for all of the python packages in the dependabot PRs. and then ran just update-reqs to update the uv.lock and .pre-commit-config.yaml files.

There was also an error in the .pre-commit-config.yaml where we should be using the hook nbdev-clean instead of nbdev_clean

How to test or reproduce? 🧪

run just venv to update your .venv/ and pre-commit hooks.

Screenshots (if applicable) 📷

Checklist ✅

  • I have run and tested my changes locally
  • I have limited this PR to less than 1000 lines of code change (if not, explain why)
  • I have updated/added tests to cover my changes (if applicable)
  • I have updated/added requirements to cover my changes (if applicable)
  • I have run linting and formatting on any code changes (if applicable)
  • I have updated the documentation (README, etc.) accordingly

Reviewer Emoji Legend

:code: Meaning
😃👍💯 :smiley: :+1: :100: I like this...

...and I want the author to know it! This is a way to highlight positive parts of a code review.
⭐⭐⭐ :star: :star: :star: Important to fix before PR can be approved...

And I am providing reasons why it needs to be addressed as well as suggested improvements.
⭐⭐ :star: :star: Important to fix but non-blocking for PR approval...

And I am providing suggestions where it could be improved either in this PR or later.
:star: Give this some thought but non-blocking for PR approval...

...and consider this a suggestion, not a requirement.
:question: I have a question.

This should be a fully formed question with sufficient information and context that requires a response.
📝 :memo: This is an explanatory note, fun fact, or relevant commentary that does not require any action.
:pick: This is a nitpick.

This does not require any changes and is often better left unsaid. This may include stylistic, formatting, or organization suggestions and should likely be prevented/enforced by linting if they really matter.
♻️ :recycle: Suggestion for refactoring.

Should include enough context to be actionable and not be considered a nitpick.

@NKeleher NKeleher requested a review from a team February 2, 2026 17:39
@claude
Copy link

claude bot commented Feb 2, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

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.

1 participant