diff options
| author | Eric N. Vander Weele <ericvw@gmail.com> | 2020-01-07 13:14:34 -0500 |
|---|---|---|
| committer | Eric N. Vander Weele <ericvw@gmail.com> | 2020-01-07 13:14:34 -0500 |
| commit | c918e7249699ab91ccbd6d69a898d19aa009c971 (patch) | |
| tree | 258f2692d8a214e48e792cdd72162d464916edc7 /src/flake8/main | |
| parent | 3d546d448a22b131f42444657f0985b37b252097 (diff) | |
| download | flake8-c918e7249699ab91ccbd6d69a898d19aa009c971.tar.gz | |
Remove unused 'isolated' parameter
Now that `ConfigFileFinder.ignore_config_files` attribute is used
everywhere and is constructed from the `--isolated` CLI option, the now
unused `isolated` parameters can be safely removed.
Diffstat (limited to 'src/flake8/main')
| -rw-r--r-- | src/flake8/main/application.py | 27 |
1 files changed, 6 insertions, 21 deletions
diff --git a/src/flake8/main/application.py b/src/flake8/main/application.py index 7ea3fbb..a30f195 100644 --- a/src/flake8/main/application.py +++ b/src/flake8/main/application.py @@ -131,8 +131,8 @@ class Application(object): (self.result_count > 0) or self.catastrophic_failure ) - def find_plugins(self, config_finder, config_file, ignore_config_files): - # type: (config.ConfigFileFinder, Optional[str], bool) -> None + def find_plugins(self, config_finder, config_file): + # type: (config.ConfigFileFinder, Optional[str]) -> None """Find and load the plugins for this application. Set the :attr:`check_plugins` and :attr:`formatting_plugins` attributes @@ -147,9 +147,7 @@ class Application(object): Determine whether to parse configuration files or not. (i.e., the --isolated option). """ - local_plugins = config.get_local_plugins( - config_finder, config_file, ignore_config_files - ) + local_plugins = config.get_local_plugins(config_finder, config_file) sys.path.extend(local_plugins.paths) @@ -173,7 +171,6 @@ class Application(object): self, config_finder, # type: config.ConfigFileFinder config_file, # type: Optional[str] - ignore_config_files, # type: bool argv, # type: List[str] ): # type: (...) -> None @@ -184,18 +181,11 @@ class Application(object): :param str config_file: The optional configuraiton file to override all other configuration files (i.e., the --config option). - :param bool ignore_config_files: - Determine whether to parse configuration files or not. (i.e., the - --isolated option). :param list argv: Command-line arguments passed in directly. """ self.options, self.args = aggregator.aggregate_options( - self.option_manager, - config_finder, - config_file, - ignore_config_files, - argv, + self.option_manager, config_finder, config_file, argv, ) self.running_against_diff = self.options.diff @@ -339,15 +329,10 @@ class Application(object): prelim_opts.append_config, ignore_config_files=prelim_opts.isolated, ) - self.find_plugins( - config_finder, prelim_opts.config, prelim_opts.isolated - ) + self.find_plugins(config_finder, prelim_opts.config) self.register_plugin_options() self.parse_configuration_and_cli( - config_finder, - prelim_opts.config, - prelim_opts.isolated, - remaining_args, + config_finder, prelim_opts.config, remaining_args, ) self.make_formatter() self.make_guide() |
