VimPlug/jedi-fork
1
0
Fork 0
forked from VimPlug/jedi
Code Pull Requests Activity

Remove use_filesystem_cache and additional_dynamic_modules, it hasn't been implemented for a long time

Browse Source
This commit is contained in:
Dave Halter
2020-01-23 23:17:26 +01:00
parent cc8a3f192d
commit 290e2151df
3 changed files with 3 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
CHANGELOG.rst
View File

@@ -31,6 +31,9 @@ Changelog
has reached the end of its life and now it's just about a smooth transition. has reached the end of its life and now it's just about a smooth transition.
- Removed ``settings.no_completion_duplicates``. It wasn't tested and nobody - Removed ``settings.no_completion_duplicates``. It wasn't tested and nobody
was probably using it anyway. was probably using it anyway.
- Removed ``settings.use_filesystem_cache`` and
``settings.additional_dynamic_modules``, they have no usage anymore. Pretty
much nobody was probably using them.
0.15.2 (2019-12-20) 0.15.2 (2019-12-20)
+++++++++++++++++++ +++++++++++++++++++

13
jedi/settings.py
View File

@@ -23,7 +23,6 @@ Filesystem cache
~~~~~~~~~~~~~~~~ ~~~~~~~~~~~~~~~~
.. autodata:: cache_directory .. autodata:: cache_directory
.. autodata:: use_filesystem_cache
Parser Parser
@@ -38,7 +37,6 @@ Dynamic stuff
.. autodata:: dynamic_array_additions .. autodata:: dynamic_array_additions
.. autodata:: dynamic_params .. autodata:: dynamic_params
.. autodata:: dynamic_params_for_other_modules .. autodata:: dynamic_params_for_other_modules
.. autodata:: additional_dynamic_modules
.. autodata:: auto_import_modules .. autodata:: auto_import_modules
@@ -71,11 +69,6 @@ Removed it again, because in VIM that is not very practical.
# Filesystem cache # Filesystem cache
# ---------------- # ----------------
use_filesystem_cache = True
"""
Use filesystem cache to save once parsed files with pickle.
"""
if platform.system().lower() == 'windows': if platform.system().lower() == 'windows':
_cache_directory = os.path.join(os.getenv('APPDATA') or '~', 'Jedi', _cache_directory = os.path.join(os.getenv('APPDATA') or '~', 'Jedi',
'Jedi') 'Jedi')
@@ -133,12 +126,6 @@ dynamic_params_for_other_modules = True
Do the same for other modules. Do the same for other modules.
""" """
additional_dynamic_modules = []
"""
Additional modules in which |jedi| checks if statements are to be found. This
is practical for IDEs, that want to administrate their modules themselves.
"""
dynamic_flow_information = True dynamic_flow_information = True
""" """
Check for `isinstance` and other information to infer a type. Check for `isinstance` and other information to infer a type.

11
test/test_settings.py
View File

@@ -23,17 +23,6 @@ def test_auto_import_modules_imports(auto_import_json, Script):
assert isinstance(main._name, CompiledValueName) assert isinstance(main._name, CompiledValueName)
def test_additional_dynamic_modules(monkeypatch, Script):
# We could add further tests, but for now it's even more important that
# this doesn't fail.
monkeypatch.setattr(
settings,
'additional_dynamic_modules',
['/foo/bar/jedi_not_existing_file.py']
)
assert not Script('def some_func(f):\n f.').complete()
def test_cropped_file_size(monkeypatch, get_names, Script): def test_cropped_file_size(monkeypatch, get_names, Script):
code = 'class Foo(): pass\n' code = 'class Foo(): pass\n'
monkeypatch.setattr( monkeypatch.setattr(