Skip to content

Commit

Permalink
Rename register_rules to collect_rules.
Browse files Browse the repository at this point in the history
# Rust tests will be skipped. Delete if not intended.
[ci skip-rust-tests]
  • Loading branch information
jsirois committed Jul 28, 2020
1 parent f12b7c0 commit 50f2893
Show file tree
Hide file tree
Showing 60 changed files with 121 additions and 121 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
from pants.engine.addresses import Addresses
from pants.engine.console import Console
from pants.engine.goal import Goal, GoalSubsystem
from pants.engine.rules import goal_rule, register_rules
from pants.engine.rules import collect_rules, goal_rule


class ListAndDieForTestingSubsystem(GoalSubsystem):
Expand All @@ -25,4 +25,4 @@ def fast_list_and_die_for_testing(console: Console, addresses: Addresses) -> Lis


def rules():
return register_rules()
return collect_rules()
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
from pants.engine.console import Console
from pants.engine.fs import Digest, MergeDigests, Workspace
from pants.engine.goal import Goal, GoalSubsystem, LineOriented
from pants.engine.rules import goal_rule, register_rules
from pants.engine.rules import collect_rules, goal_rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import FieldSet, TargetsToValidFieldSets, TargetsToValidFieldSetsRequest
from pants.engine.unions import union
Expand Down Expand Up @@ -79,4 +79,4 @@ async def create_awslambda(


def rules():
return register_rules()
return collect_rules()
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
from pants.core.util_rules import strip_source_roots
from pants.engine.fs import Digest, MergeDigests
from pants.engine.process import Process, ProcessResult
from pants.engine.rules import register_rules, rule
from pants.engine.rules import collect_rules, rule
from pants.engine.selectors import Get
from pants.engine.unions import UnionRule
from pants.python.python_setup import PythonSetup
Expand Down Expand Up @@ -129,7 +129,7 @@ async def setup_lambdex(lambdex: Lambdex) -> LambdexSetup:

def rules():
return [
*register_rules(),
*collect_rules(),
UnionRule(AWSLambdaFieldSet, PythonAwsLambdaFieldSet),
*download_pex_bin.rules(),
*python_sources.rules(),
Expand Down
4 changes: 2 additions & 2 deletions src/python/pants/backend/codegen/protobuf/python/rules.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
from pants.engine.fs import AddPrefix, Digest, MergeDigests, RemovePrefix, Snapshot
from pants.engine.platform import Platform
from pants.engine.process import Process, ProcessResult
from pants.engine.rules import register_rules, rule
from pants.engine.rules import collect_rules, rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import GeneratedSources, GenerateSourcesRequest, Sources, TransitiveTargets
from pants.engine.unions import UnionRule
Expand Down Expand Up @@ -121,6 +121,6 @@ async def generate_python_from_protobuf(

def rules():
return [
*register_rules(),
*collect_rules(),
UnionRule(GenerateSourcesRequest, GeneratePythonFromProtobufRequest),
]
4 changes: 2 additions & 2 deletions src/python/pants/backend/codegen/protobuf/target_types.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

from pants.backend.codegen.protobuf.protoc import Protoc
from pants.engine.addresses import Address
from pants.engine.rules import register_rules, rule
from pants.engine.rules import collect_rules, rule
from pants.engine.target import (
COMMON_TARGET_FIELDS,
Dependencies,
Expand Down Expand Up @@ -51,6 +51,6 @@ def inject_dependencies(_: InjectProtobufDependencies, protoc: Protoc) -> Inject

def rules():
return [
*register_rules(),
*collect_rules(),
UnionRule(InjectDependenciesRequest, InjectProtobufDependencies),
]
4 changes: 2 additions & 2 deletions src/python/pants/backend/pants_info/list_target_types.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
from pants.core.util_rules.pants_bin import PantsBin
from pants.engine.console import Console
from pants.engine.goal import Goal, GoalSubsystem, LineOriented
from pants.engine.rules import goal_rule, register_rules
from pants.engine.rules import collect_rules, goal_rule
from pants.engine.target import (
AsyncField,
BoolField,
Expand Down Expand Up @@ -249,4 +249,4 @@ def list_target_types(


def rules():
return register_rules()
return collect_rules()
4 changes: 2 additions & 2 deletions src/python/pants/backend/project_info/cloc.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
from pants.engine.goal import Goal, GoalSubsystem
from pants.engine.platform import Platform
from pants.engine.process import Process, ProcessResult
from pants.engine.rules import goal_rule, register_rules
from pants.engine.rules import collect_rules, goal_rule
from pants.engine.selectors import Get
from pants.util.strutil import pluralize

Expand Down Expand Up @@ -131,4 +131,4 @@ async def run_cloc(


def rules():
return register_rules()
return collect_rules()
4 changes: 2 additions & 2 deletions src/python/pants/backend/project_info/dependees.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
from pants.engine.collection import DeduplicatedCollection
from pants.engine.console import Console
from pants.engine.goal import Goal, GoalSubsystem, LineOriented
from pants.engine.rules import goal_rule, register_rules, rule
from pants.engine.rules import collect_rules, goal_rule, rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import Dependencies, DependenciesRequest, Targets
from pants.util.frozendict import FrozenDict
Expand Down Expand Up @@ -182,4 +182,4 @@ async def dependees_goal(


def rules():
return register_rules()
return collect_rules()
4 changes: 2 additions & 2 deletions src/python/pants/backend/project_info/dependencies.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
from pants.engine.addresses import Addresses
from pants.engine.console import Console
from pants.engine.goal import Goal, GoalSubsystem, LineOriented
from pants.engine.rules import goal_rule, register_rules
from pants.engine.rules import collect_rules, goal_rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import Dependencies as DependenciesField
from pants.engine.target import DependenciesRequest, Targets, TransitiveTargets
Expand Down Expand Up @@ -104,4 +104,4 @@ async def dependencies(


def rules():
return register_rules()
return collect_rules()
4 changes: 2 additions & 2 deletions src/python/pants/backend/project_info/filedeps.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
from pants.engine.addresses import Address, Addresses, BuildFileAddress
from pants.engine.console import Console
from pants.engine.goal import Goal, GoalSubsystem, LineOriented
from pants.engine.rules import goal_rule, register_rules
from pants.engine.rules import collect_rules, goal_rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import (
HydratedSources,
Expand Down Expand Up @@ -120,4 +120,4 @@ async def file_deps(


def rules():
return register_rules()
return collect_rules()
4 changes: 2 additions & 2 deletions src/python/pants/backend/project_info/filter_targets.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
from pants.base.deprecated import resolve_conflicting_options
from pants.engine.console import Console
from pants.engine.goal import Goal, GoalSubsystem, LineOriented
from pants.engine.rules import goal_rule, register_rules
from pants.engine.rules import collect_rules, goal_rule
from pants.engine.target import (
RegisteredTargetTypes,
Tags,
Expand Down Expand Up @@ -162,4 +162,4 @@ def filter_tag_regex(tag_regex: str) -> TargetFilter:


def rules():
return register_rules()
return collect_rules()
4 changes: 2 additions & 2 deletions src/python/pants/backend/project_info/list_roots.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

from pants.engine.console import Console
from pants.engine.goal import Goal, GoalSubsystem, LineOriented
from pants.engine.rules import goal_rule, register_rules
from pants.engine.rules import collect_rules, goal_rule
from pants.source.source_root import AllSourceRoots


Expand All @@ -28,4 +28,4 @@ async def list_roots(


def rules():
return register_rules()
return collect_rules()
4 changes: 2 additions & 2 deletions src/python/pants/backend/project_info/list_targets.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
from pants.engine.addresses import Address, Addresses
from pants.engine.console import Console
from pants.engine.goal import Goal, GoalSubsystem, LineOriented
from pants.engine.rules import goal_rule, register_rules
from pants.engine.rules import collect_rules, goal_rule
from pants.engine.selectors import Get
from pants.engine.target import DescriptionField, ProvidesField, Targets

Expand Down Expand Up @@ -111,4 +111,4 @@ async def list_targets(


def rules():
return register_rules()
return collect_rules()
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
from pants.engine.console import Console
from pants.engine.fs import Digest, DigestContents, SourcesSnapshot
from pants.engine.goal import Goal, GoalSubsystem
from pants.engine.rules import goal_rule, register_rules
from pants.engine.rules import collect_rules, goal_rule
from pants.engine.selectors import Get
from pants.subsystem.subsystem import Subsystem
from pants.util.frozendict import FrozenDict
Expand Down Expand Up @@ -334,4 +334,4 @@ async def validate(


def rules():
return register_rules()
return collect_rules()
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
StripSourcesFieldRequest,
)
from pants.engine.addresses import Address
from pants.engine.rules import register_rules, rule
from pants.engine.rules import collect_rules, rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import (
HydratedSources,
Expand Down Expand Up @@ -166,4 +166,4 @@ async def map_module_to_address(


def rules():
return register_rules()
return collect_rules()
4 changes: 2 additions & 2 deletions src/python/pants/backend/python/dependency_inference/rules.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
)
from pants.engine.fs import Digest, DigestContents
from pants.engine.internals.graph import Owners, OwnersRequest
from pants.engine.rules import register_rules, rule
from pants.engine.rules import collect_rules, rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import (
HydratedSources,
Expand Down Expand Up @@ -158,7 +158,7 @@ async def infer_python_conftest_dependencies(

def rules():
return [
*register_rules(),
*collect_rules(),
*inject_ancestor_files.rules(),
*module_mapper.rules(),
UnionRule(InferDependenciesRequest, InferPythonDependencies),
Expand Down
4 changes: 2 additions & 2 deletions src/python/pants/backend/python/lint/bandit/rules.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
)
from pants.engine.fs import Digest, GlobMatchErrorBehavior, MergeDigests, PathGlobs
from pants.engine.process import FallibleProcessResult, Process
from pants.engine.rules import register_rules, rule
from pants.engine.rules import collect_rules, rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import FieldSetWithOrigin
from pants.engine.unions import UnionRule
Expand Down Expand Up @@ -152,7 +152,7 @@ async def bandit_lint(

def rules():
return [
*register_rules(),
*collect_rules(),
UnionRule(LintRequest, BanditRequest),
*download_pex_bin.rules(),
*determine_source_files.rules(),
Expand Down
4 changes: 2 additions & 2 deletions src/python/pants/backend/python/lint/black/rules.py
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
)
from pants.engine.fs import EMPTY_SNAPSHOT, Digest, GlobMatchErrorBehavior, MergeDigests, PathGlobs
from pants.engine.process import FallibleProcessResult, Process, ProcessResult
from pants.engine.rules import register_rules, rule
from pants.engine.rules import collect_rules, rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import FieldSetWithOrigin
from pants.engine.unions import UnionRule
Expand Down Expand Up @@ -188,7 +188,7 @@ async def black_lint(field_sets: BlackRequest, black: Black) -> LintResults:

def rules():
return [
*register_rules(),
*collect_rules(),
UnionRule(PythonFmtRequest, BlackRequest),
UnionRule(LintRequest, BlackRequest),
*download_pex_bin.rules(),
Expand Down
4 changes: 2 additions & 2 deletions src/python/pants/backend/python/lint/docformatter/rules.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
)
from pants.engine.fs import EMPTY_SNAPSHOT, Digest, MergeDigests
from pants.engine.process import FallibleProcessResult, Process, ProcessResult
from pants.engine.rules import register_rules, rule
from pants.engine.rules import collect_rules, rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import FieldSetWithOrigin
from pants.engine.unions import UnionRule
Expand Down Expand Up @@ -162,7 +162,7 @@ async def docformatter_lint(

def rules():
return [
*register_rules(),
*collect_rules(),
UnionRule(PythonFmtRequest, DocformatterRequest),
UnionRule(LintRequest, DocformatterRequest),
*download_pex_bin.rules(),
Expand Down
4 changes: 2 additions & 2 deletions src/python/pants/backend/python/lint/flake8/rules.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
Snapshot,
)
from pants.engine.process import FallibleProcessResult, Process
from pants.engine.rules import register_rules, rule
from pants.engine.rules import collect_rules, rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import FieldSetWithOrigin
from pants.engine.unions import UnionRule
Expand Down Expand Up @@ -190,7 +190,7 @@ async def flake8_lint(

def rules():
return [
*register_rules(),
*collect_rules(),
UnionRule(LintRequest, Flake8Request),
*download_pex_bin.rules(),
*determine_source_files.rules(),
Expand Down
4 changes: 2 additions & 2 deletions src/python/pants/backend/python/lint/isort/rules.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
PathGlobs,
)
from pants.engine.process import FallibleProcessResult, Process, ProcessResult
from pants.engine.rules import register_rules, rule
from pants.engine.rules import collect_rules, rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import FieldSetWithOrigin
from pants.engine.unions import UnionRule
Expand Down Expand Up @@ -192,7 +192,7 @@ async def isort_lint(request: IsortRequest, isort: Isort) -> LintResults:

def rules():
return [
*register_rules(),
*collect_rules(),
UnionRule(PythonFmtRequest, IsortRequest),
UnionRule(LintRequest, IsortRequest),
*download_pex_bin.rules(),
Expand Down
4 changes: 2 additions & 2 deletions src/python/pants/backend/python/lint/pylint/rules.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
PathGlobs,
)
from pants.engine.process import FallibleProcessResult, Process
from pants.engine.rules import register_rules, rule
from pants.engine.rules import collect_rules, rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import (
Dependencies,
Expand Down Expand Up @@ -310,7 +310,7 @@ async def pylint_lint(

def rules():
return [
*register_rules(),
*collect_rules(),
UnionRule(LintRequest, PylintRequest),
*download_pex_bin.rules(),
*determine_source_files.rules(),
Expand Down
4 changes: 2 additions & 2 deletions src/python/pants/backend/python/lint/python_fmt.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
from pants.core.goals.style_request import StyleRequest
from pants.core.util_rules.determine_source_files import AllSourceFilesRequest, SourceFiles
from pants.engine.fs import Digest, Snapshot
from pants.engine.rules import register_rules, rule
from pants.engine.rules import collect_rules, rule
from pants.engine.selectors import Get
from pants.engine.unions import UnionMembership, UnionRule, union

Expand Down Expand Up @@ -66,4 +66,4 @@ async def format_python_target(


def rules():
return [*register_rules(), UnionRule(LanguageFmtTargets, PythonFmtTargets)]
return [*collect_rules(), UnionRule(LanguageFmtTargets, PythonFmtTargets)]
4 changes: 2 additions & 2 deletions src/python/pants/backend/python/rules/coverage.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
PathGlobs,
)
from pants.engine.process import Process, ProcessResult
from pants.engine.rules import register_rules, rule
from pants.engine.rules import collect_rules, rule
from pants.engine.selectors import Get, MultiGet
from pants.engine.target import TransitiveTargets
from pants.engine.unions import UnionRule
Expand Down Expand Up @@ -335,6 +335,6 @@ def _get_coverage_reports(

def rules():
return [
*register_rules(),
*collect_rules(),
UnionRule(CoverageDataCollection, PytestCoverageDataCollection),
]

0 comments on commit 50f2893

Please sign in to comment.