Skip to content

fix skip with package = wheel #3269

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

Merged
merged 10 commits into from
Apr 27, 2024
Merged

Conversation

MarcinKonowalczyk
Copy link
Contributor

@MarcinKonowalczyk MarcinKonowalczyk commented Apr 21, 2024

Fix for #3153

  • ran the linter to address style issues (tox -e fix)
  • wrote descriptive pull request text
  • ensured there are test(s) validating the fix
  • added news fragment in docs/changelog folder
  • [n/a] updated/extended the documentation

Sorry, something went wrong.

Copy link
Member

@gaborbernat gaborbernat left a comment

Choose a reason for hiding this comment

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

Test please

@gaborbernat gaborbernat marked this pull request as draft April 23, 2024 16:43

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@MarcinKonowalczyk
Copy link
Contributor Author

Ok, the only failing test is test_depends.py due to missing output lines:
Screenshot 2024-04-26 at 18 28 13

Any hint as to where am I supposed to return that status? (sorry, a bit lost in the code) so far i'm looking at report called from execute, both in common.py

@MarcinKonowalczyk
Copy link
Contributor Author

For reference, here is the stack from the failing case:

Traceback (most recent call last):
  File ".../tox/src/tox/run.py", line 20, in run
    result = main(sys.argv[1:] if args is None else args)
  File ".../tox/src/tox/run.py", line 46, in main
    return handler(state)
  File ".../tox/src/tox/session/cmd/legacy.py", line 115, in legacy
    return run_sequential(state)
  File ".../tox/src/tox/session/cmd/run/sequential.py", line 25, in run_sequential
    return execute(state, max_workers=1, has_spinner=False, live=True)
  File ".../tox/src/tox/session/cmd/run/common.py", line 170, in execute
    cast(RunToxEnv, state.envs[name]).mark_active()
  File ".../tox/src/tox/tox_env/runner.py", line 211, in mark_active
    for pkg_env in self.package_envs:
  File ".../tox/src/tox/tox_env/runner.py", line 208, in package_envs
    yield from self.package_env.child_pkg_envs(self.conf)
  File ".../tox/src/tox/tox_env/python/package.py", line 124, in child_pkg_envs
    conf = run_conf["wheel_build_env"]
  File ".../tox/src/tox/config/sets.py", line 116, in __getitem__
    return self.load(item)
  File ".../tox/src/tox/config/sets.py", line 127, in load
    return config_definition.__call__(self._conf, self.loaders, ConfigLoadArgs(chain, self.name, self.env_name))  # noqa: PLC2801
  File ".../tox/src/tox/config/of_type.py", line 111, in __call__
    value = self.default(conf, args.env_name) if callable(self.default) else self.default
  File ".../tox/src/tox/tox_env/python/package.py", line 93, in default_wheel_tag
    run_py = cast(Python, run_env).base_python
  File ".../tox/src/tox/tox_env/python/api.py", line 260, in base_python
    raise Skip(msg)
tox.tox_env.errors.Skip: could not find python interpreter with spec(s): py310

MarcinKonowalczyk and others added 5 commits April 27, 2024 12:50

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
pre-commit-ci bot and others added 2 commits April 27, 2024 11:55
for more information, see https://pre-commit.ci
@MarcinKonowalczyk
Copy link
Contributor Author

ok, compared and matched the behaviour to package = sdist. After stepping through the code it looks like it was nothing to do with the report, which looks correct when running the normal tox command with the patch on a test repo.

@MarcinKonowalczyk
Copy link
Contributor Author

MarcinKonowalczyk commented Apr 27, 2024

Also patched _make_tox_wheel to allow pytest --run-integration run with a dev patch version, to allow all tests to pass locally.

@MarcinKonowalczyk MarcinKonowalczyk marked this pull request as ready for review April 27, 2024 12:34
@gaborbernat gaborbernat merged commit 3db9822 into tox-dev:main Apr 27, 2024
25 checks passed
@gaborbernat
Copy link
Member

Thank you for your contribution.

naa0yama referenced this pull request in naa0yama/ffvqe Jul 31, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [tox](https://togithub.com/tox-dev/tox)
([changelog](https://tox.wiki/en/latest/changelog.html)) | `4.15.0` ->
`4.15.1` |
[![age](https://developer.mend.io/api/mc/badges/age/pypi/tox/4.15.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/tox/4.15.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/tox/4.15.0/4.15.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/tox/4.15.0/4.15.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>tox-dev/tox (tox)</summary>

### [`v4.15.1`](https://togithub.com/tox-dev/tox/releases/tag/4.15.1)

[Compare
Source](https://togithub.com/tox-dev/tox/compare/4.15.0...4.15.1)

<!-- Release notes generated using configuration in .github/release.yml
at 4.15.1 -->

#### What's Changed

- fix skip with package = wheel by
[@&#8203;MarcinKonowalczyk](https://togithub.com/MarcinKonowalczyk) in
[https://github.com/tox-dev/tox/pull/3269](https://togithub.com/tox-dev/tox/pull/3269)
- Fixed typo in user guide. by
[@&#8203;carltongibson](https://togithub.com/carltongibson) in
[https://github.com/tox-dev/tox/pull/3277](https://togithub.com/tox-dev/tox/pull/3277)
- Fix broad build privileges @&#8203; GHA release workflow by
[@&#8203;webknjaz](https://togithub.com/webknjaz) in
[https://github.com/tox-dev/tox/pull/3281](https://togithub.com/tox-dev/tox/pull/3281)
- Allow `ConfigSet.add_config` to receive parameterized generics for
`of_type`. by [@&#8203;ssbarnea](https://togithub.com/ssbarnea) in
[https://github.com/tox-dev/tox/pull/3288](https://togithub.com/tox-dev/tox/pull/3288)
- Fix section substitution with setenv by
[@&#8203;JJLLWW](https://togithub.com/JJLLWW) in
[https://github.com/tox-dev/tox/pull/3289](https://togithub.com/tox-dev/tox/pull/3289)

#### New Contributors

- [@&#8203;MarcinKonowalczyk](https://togithub.com/MarcinKonowalczyk)
made their first contribution in
[https://github.com/tox-dev/tox/pull/3269](https://togithub.com/tox-dev/tox/pull/3269)
- [@&#8203;carltongibson](https://togithub.com/carltongibson) made their
first contribution in
[https://github.com/tox-dev/tox/pull/3277](https://togithub.com/tox-dev/tox/pull/3277)
- [@&#8203;JJLLWW](https://togithub.com/JJLLWW) made their first
contribution in
[https://github.com/tox-dev/tox/pull/3289](https://togithub.com/tox-dev/tox/pull/3289)

**Full Changelog**:
tox-dev/tox@4.15.0...4.15.1

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View the
[repository job
log](https://developer.mend.io/github/naa0yama/FFmpeg-video-quality-evaluations).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zODguMSIsInVwZGF0ZWRJblZlciI6IjM3LjQzOC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJkYXRhc291cmNlOnB5cGkiLCJkZXBUeXBlOmRldiIsIm1hbmFnZXI6cG9ldHJ5IiwicmVub3ZhdGUiLCJ1cGRhdGUtcGF0Y2giLCJ2ZXJzaW9uaW5nOnBvZXRyeSJdfQ==-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants