Skip to content

Commit

Permalink
fix: --only shouldn't require prerelease pythons setting (#1564)
Browse files Browse the repository at this point in the history
* fix: --only shouldn't require prerelease pythons setting

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>

* Add test

---------

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
Co-authored-by: Joe Rickerby <joerick@mac.com>
  • Loading branch information
henryiii and joerick authored Aug 14, 2023
1 parent 12d5488 commit ce71f44
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions cibuildwheel/options.py
Original file line number Diff line number Diff line change
Expand Up @@ -450,6 +450,7 @@ def globals(self) -> GlobalOptions:
build_config = args.only
skip_config = ""
architectures = Architecture.all_archs(self.platform)
prerelease_pythons = True

build_selector = BuildSelector(
build_config=build_config,
Expand Down
1 change: 1 addition & 0 deletions unit_test/main_tests/main_platform_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -214,6 +214,7 @@ def test_only_argument(intercepted_build_args, monkeypatch, only, plat):
assert options.globals.build_selector.skip_config == ""
assert options.platform == plat
assert options.globals.architectures == Architecture.all_archs(plat)
assert options.globals.build_selector.prerelease_pythons is True


@pytest.mark.parametrize("only", ("cp311-manylxinux_x86_64", "some_linux_thing"))
Expand Down

0 comments on commit ce71f44

Please sign in to comment.