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

@paikendpaikend 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

@S0okJuS0okJu left a comment

Choose a reason for hiding this comment

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

LGTM

@choieastseachoieastsea merged commit fa77a53 into developOct 8, 2025
6 checks passed
@choieastseachoieastsea 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 freeto 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

@choieastsea@paikend@S0okJu