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

remove duplicate header in fairchem-data-om/pyproject.toml #917

Merged

Conversation

IliasChair
Copy link
Contributor

The pyproject.toml file from the fairchem-data-om package contains two instances of the [tool.hatch.build] header. This causes the following error when running ruff check:

ruff failed
  Cause: Failed to parse fairchem/packages/fairchem-data-om/pyproject.toml
  Cause: TOML parse error at line 33, column 1
   |
33 | [tool.hatch.build]
   | ^
invalid table header
duplicate key `build` in table `tool.hatch`

Kept the first header and directory, since ../../dist-data-om matches as the other package configurations.

@rayg1234 rayg1234 requested a review from lbluque November 19, 2024 18:56
@rayg1234
Copy link
Collaborator

@lbluque can you take a quick look?

Copy link

codecov bot commented Nov 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅


🚨 Try these New Features:

@misko misko added enhancement New feature or request minor Minor version release labels Nov 19, 2024
@lbluque
Copy link
Collaborator

lbluque commented Nov 20, 2024

Thanks @IliasChair !

@lbluque lbluque enabled auto-merge November 20, 2024 03:29
@misko misko self-requested a review November 21, 2024 21:31
@lbluque lbluque added this pull request to the merge queue Nov 21, 2024
Merged via the queue into FAIR-Chem:main with commit 38a4bb3 Nov 21, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request minor Minor version release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants