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 the concept of a scope category. #10224

Merged
merged 20 commits into from Jul 3, 2020

Conversation

benjyw
Copy link
Sponsor Contributor

@benjyw benjyw commented Jul 1, 2020

In v2 all optionables are subsystems, even global options.
There is no meaning to categories any more.

[ci skip-rust-tests]

In v2 all optionables are subsystems, even global options.
There is no meaning to categories any more.

[ci skip-rust-tests]
# Rust tests will be skipped. Delete if not intended.
[ci skip-rust-tests]
Copy link
Contributor

@Eric-Arellano Eric-Arellano left a comment

Choose a reason for hiding this comment

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

Awesome!

@@ -54,22 +50,9 @@ def deprecated_cache_setup_removal_version(cls):
"""
return None

@classmethod
def conflict_free_name(cls):
Copy link
Contributor

Choose a reason for hiding this comment

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

By the way, v2 setup-py is still setup-py2:

name = "setup-py2"
@classmethod
def register_options(cls, register):
super().register_options(register)
register(
"--args",
type=list,
member_type=shell_str,
passthrough=True,
help="Arguments to pass directly to setup.py, e.g. "
'`--setup-py2-args="bdist_wheel --python-tag py36.py37"`. If unspecified, Pants will '
"dump the setup.py chroot.",
)

# Rust tests will be skipped. Delete if not intended.
[ci skip-rust-tests]
Copy link
Contributor

@Eric-Arellano Eric-Arellano left a comment

Choose a reason for hiding this comment

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

Yay to removing ScopeInfoIterator!

benjyw added 17 commits July 1, 2020 12:00
# Rust tests will be skipped. Delete if not intended.
[ci skip-rust-tests]
Instead use --plugins/--backend-packages, as there is
now no v1 to get in the way.

[ci skip-rust-tests]
[ci skip-rust-tests]
# Rust tests will be skipped. Delete if not intended.
[ci skip-rust-tests]
# Rust tests will be skipped. Delete if not intended.
[ci skip-rust-tests]
# Rust tests will be skipped. Delete if not intended.
[ci skip-rust-tests]
# Rust tests will be skipped. Delete if not intended.
[ci skip-rust-tests]
# Rust tests will be skipped. Delete if not intended.
[ci skip-rust-tests]
# Rust tests will be skipped. Delete if not intended.
[ci skip-rust-tests]
# Rust tests will be skipped. Delete if not intended.
[ci skip-rust-tests]
@benjyw benjyw merged commit 8500384 into pantsbuild:master Jul 3, 2020
@benjyw benjyw deleted the remove_scope_category branch July 3, 2020 00:04
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