Skip to content

chore: Move requirements file under tools/ - #182

Merged
bsipocz merged 1 commit into
scientific-python:mainfrom
matthewfeickert:chore/move-tooling
Sep 3, 2026
Merged

chore: Move requirements file under tools/#182
bsipocz merged 1 commit into
scientific-python:mainfrom
matthewfeickert:chore/move-tooling

Conversation

@matthewfeickert

Copy link
Copy Markdown
Member
  • As the requirements.txt exists only for the stale-wheels updater GitHub Action, move this under the tools/ directory to have the top level directory space be disambiguated.

This is purely cosmetic.

* As the requirements.txt exists only for the stale-wheels updater GitHub
  Action, move this under the tools/ directory to have the top level directory
  space be disambiguated.

@bsipocz bsipocz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

OK, makes sense. Thanks!

@bsipocz bsipocz added the chore label Sep 3, 2026
@bsipocz

bsipocz commented Sep 3, 2026

Copy link
Copy Markdown
Member

unrelated, but what about renaming the "chore" label to "maintenance"?

@bsipocz
bsipocz merged commit fd44fde into scientific-python:main Sep 3, 2026
4 checks passed
@larsoner

larsoner commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

I suspect this broke dependabot updates since that looks in / not /tools/. I'll add a fix to #178

@matthewfeickert
matthewfeickert deleted the chore/move-tooling branch September 3, 2026 17:40
@matthewfeickert

Copy link
Copy Markdown
Member Author

I suspect this broke dependabot updates since that looks in / not /tools/.

Ah yeah, correct as the reference docs (https://docs.github.com/en/code-security/reference/supply-chain-security/dependabot-options-reference) show that it doesn't traverse down

updates:
  - package-ecosystem: ...
    # Look for package and lock files in the `root` directory
    directory: "/"

I'll add a fix to #178

Thanks!

@bsipocz

bsipocz commented Sep 3, 2026

Copy link
Copy Markdown
Member

My bad, sorry for not catching it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants