Skip to content

Sym two #814

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.

Sign up for GitHub

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

Merged
thomasballinger merged 24 commits into from
Jul 22, 2020
Merged

Sym two #814

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
f4c0ed6
some changes to file reading
etiennerichart Jun 25, 2020
347707d
Some work on symbolic link files
etiennerichart Jun 26, 2020
3f13a28
Cleaned up symlink fix ideas
etiennerichart Jun 26, 2020
47ea5ee
More clean up
etiennerichart Jun 26, 2020
6eff3b2
Make it easier to test different methods
etiennerichart Jun 26, 2020
8e08f3a
Found an even better way. useNameSubname = 4
etiennerichart Jun 26, 2020
2b96e8d
"Used os.path.realpath to fix issue"
etiennerichart Jun 29, 2020
44f76f7
Removed code that I was using for testing
etiennerichart Jun 29, 2020
513f37d
Made variable to store paths for comparison.
etiennerichart Jun 30, 2020
01a6a4c
further finalize changes to importcompletion.py
etiennerichart Jul 3, 2020
2edc977
Working on test but am having issues with the try
etiennerichart Jul 6, 2020
62bed48
replaced import test file with them file directory
etiennerichart Jul 7, 2020
45ada74
fixed importtest.py
etiennerichart Jul 8, 2020
9fafb2e
moved and renamed importtest to test folder
etiennerichart Jul 8, 2020
52cef9f
cleaned up test file
etiennerichart Jul 9, 2020
6983428
improved test
etiennerichart Jul 9, 2020
5928198
made it so test skips in python 2
etiennerichart Jul 9, 2020
dbc1d4f
made skipping work
etiennerichart Jul 9, 2020
5f15d7d
removed chdir and made better test skip
etiennerichart Jul 13, 2020
8bd85e9
fix symbolic links for the test
etiennerichart Jul 16, 2020
0a29731
used black auto format
etiennerichart Jul 16, 2020
cec627a
improved test to work on other computers
etiennerichart Jul 16, 2020
629bf1e
made testing strickter
etiennerichart Jul 16, 2020
11369fc
further testing
etiennerichart Jul 16, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Made variable to store paths for comparison.
  • Loading branch information
@etiennerichart
etiennerichart committed Jun 30, 2020
commit 513f37d45c48464297d3fa792ede5e7358d820bb
8 changes: 4 additions & 4 deletions bpython/importcompletion.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,8 @@

# The cached list of all known modules
modules = set()
#List of stored paths
Comment thread
sebastinas marked this conversation as resolved.
Outdated
paths = set()
fully_loaded = False


Expand Down Expand Up @@ -192,11 +194,9 @@ def find_modules(path):
continue
else:
if is_package:
pathGood = True
pathReal = os.path.realpath(pathname)
Copy link
Copy Markdown
Member

@thomasballinger thomasballinger Jul 1, 2020

Choose a reason for hiding this comment

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

small nit, but in this project (and most Python projects) we use snake_case instead of camelCase

if len(pathReal) < len(pathname) and pathReal in pathname:
pathGood = False
if pathGood:
if not pathReal in paths:
Copy link
Copy Markdown
Member

@thomasballinger thomasballinger Jul 1, 2020

Choose a reason for hiding this comment

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

in Python not needle in haystack is parsed as (not needle) in haystack, so

>>> not "/some/path"
False
>>> not "/some/path" in ["/some/path"]
False
>>> False in [False]
True
>>> not "/some/path" in [False]
True

so instead, this should be
if pathReal not in paths:

paths.add(pathReal)
for subname in find_modules(pathname):
if subname != "__init__":
yield "%s.%s" % (name, subname)
Expand Down