Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch metadata to pyproject #89

Merged
merged 4 commits into from
Jul 22, 2024
Merged

Switch metadata to pyproject #89

merged 4 commits into from
Jul 22, 2024

Conversation

alexfikl
Copy link
Contributor

This finishes up moving the project metadata to pyproject.toml and fixes the remaining ruff rules.

Comment on lines +279 to +284
"_BOXTREE_TEST_NAME": "constantone",
"_BOXTREE_TEST_DIMS": dims,
"_BOXTREE_TEST_NSOURCES": nsources,
"_BOXTREE_TEST_NTARGETS": ntargets,
"_BOXTREE_TEST_TMP_CACHE_BASEDIR": tmp_path / "boxtree_distributed_test",
"_BOXTREE_TEST_MPOLES_ALLREDUCE": False
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Those SIM rules have a bunch of interesting demands
https://docs.astral.sh/ruff/rules/uncapitalized-environment-variables/

@alexfikl alexfikl marked this pull request as draft July 22, 2024 06:21
@alexfikl alexfikl force-pushed the pyproject branch 2 times, most recently from 3b1e285 to 6f88156 Compare July 22, 2024 06:42
@alexfikl alexfikl marked this pull request as ready for review July 22, 2024 10:44
@inducer inducer merged commit 134b37d into inducer:main Jul 22, 2024
8 checks passed
@inducer
Copy link
Owner

inducer commented Jul 22, 2024

Thx!

@alexfikl alexfikl deleted the pyproject branch July 22, 2024 14:52
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