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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

unify config in pyproject.toml #1520

Closed
wants to merge 4 commits into from
Closed

unify config in pyproject.toml #1520

wants to merge 4 commits into from

Conversation

Borda
Copy link

@Borda Borda commented Jun 12, 2023

No description provided.

@Borda Borda marked this pull request as draft June 12, 2023 14:34
@Borda
Copy link
Author

Borda commented Jun 12, 2023

well this is strange; no idea where this comes from :(

ValueError: option names {'--run-podman'} already added
Traceback (most recent call last):
  File "/tmp/cirrus-ci-build/./bin/run_tests.py", line 31, in <module>
    subprocess.run(unit_test_args, check=True)
  File "/usr/lib/python3.10/subprocess.py", line 524, in run
    raise CalledProcessError(retcode, process.args,

@Borda
Copy link
Author

Borda commented Jun 12, 2023

Also, seem that not all arguments are supported by various CI platforms... 馃Ι
any thought or suggestion @joerick | @henryiii

@henryiii
Copy link
Contributor

henryiii commented Jun 12, 2023

Already trying this sort of thing in #1297.

You need to follow PEP 621, not just copy values over. Several tools can do this for you. I use pipx run hatch new --init to convert.

@Borda
Copy link
Author

Borda commented Jun 12, 2023

You need to follow PEP 621, not just copy values over. Several tools can do this for you. I use pipx run hatch new --init to convert.

I did create it by following the official docs...

Already trying this sort of thing in #1297.

ok, then I ll close this one as duplicate 馃Ι

@Borda Borda closed this Jun 12, 2023
@Borda Borda deleted the pyproject branch June 12, 2023 22:54
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.

None yet

2 participants