Skip to content

Conversation

@choieastsea
Copy link
Collaborator

Overview

I fixed pre-commit-config.yaml so that we can provide requirements.txt and requirements-dev.txt automatically.

Key Changes

  • Initialize txt files (with uv export)
  • add uv-export job to pre-commit config file
  • Thanks to uv-pre-commit, I just simply added the hook.

Related Issues

Additional context

I think there are 2 ways that we can provide dependency requirements specifications automatically, github ci or pre-commit.

Since github ci might add a separate commit, I chose the latter.

Feel free to give feedback!

Copy link
Collaborator

@paikend paikend left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@S0okJu S0okJu left a comment

Choose a reason for hiding this comment

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

LGTM

@choieastsea choieastsea merged commit fa77a53 into develop Oct 8, 2025
6 checks passed
@choieastsea choieastsea deleted the fix/pre-commit-requirements branch October 11, 2025 03:09
halucinor pushed a commit that referenced this pull request Oct 23, 2025
* deploy: pre-commit

* add `requirements.txt` and `requirements-dev.txt`
halucinor pushed a commit that referenced this pull request Oct 24, 2025
* deploy: pre-commit

* add `requirements.txt` and `requirements-dev.txt`
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.

4 participants