Skip to content

Changed dict key-matching regex to capture any valid dict key #920

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 13 commits into from
Oct 6, 2021
Merged

Changed dict key-matching regex to capture any valid dict key #920

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
eb321aa
Changed dict key-matching regex to capture any valid dict key
arian-deimling Sep 27, 2021
3f20d8e
dict key matching regex no longer matches beyond the end of a key
arian-deimling Sep 28, 2021
97d6b19
Updated regex to handle str, bytes, int, float, tuple dict keys
arian-deimling Sep 29, 2021
5bfd09c
Added tests for regex changes in prior commit
arian-deimling Sep 29, 2021
e470176
removed erroneous print()
arian-deimling Sep 29, 2021
5299f2b
Added comments to regex using re.VERBOSE flag
arian-deimling Sep 29, 2021
12ff50d
Removed unnecessary regular expressions
arian-deimling Sep 29, 2021
d57f008
Removed old commented-out code
arian-deimling Sep 29, 2021
a0f7fae
minor formatting changes to pass black linter test
arian-deimling Sep 29, 2021
236dc0b
formatting change; adjusted single-quote string regex; added tests
arian-deimling Oct 1, 2021
286d71c
modified test to test strings containing '[' and ']'
arian-deimling Oct 1, 2021
01ed93b
added test case for string dict key being typed inside []
arian-deimling Oct 1, 2021
0a08f41
ran black formatter on file
arian-deimling Oct 1, 2021
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
Next Next commit
Changed dict key-matching regex to capture any valid dict key
  • Loading branch information
@arian-deimling
arian-deimling committed Sep 27, 2021
commit eb321aa9ec088eb4a97ef61ab1bc6a9bb10dfb03
4 changes: 2 additions & 2 deletions bpython/line.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ def current_word(cursor_offset: int, line: str) -> Optional[LinePart]:
return LinePart(start, end, word)


_current_dict_key_re = LazyReCompile(r"""[\w_][\w0-9._]*\[([\w0-9._(), '"]*)""")
_current_dict_key_re = LazyReCompile(r"""[\w_][\w0-9._]*\[(.*)""")


def current_dict_key(cursor_offset: int, line: str) -> Optional[LinePart]:
Expand All @@ -45,7 +45,7 @@ def current_dict_key(cursor_offset: int, line: str) -> Optional[LinePart]:
return None


_current_dict_re = LazyReCompile(r"""([\w_][\w0-9._]*)\[([\w0-9._(), '"]*)""")
_current_dict_re = LazyReCompile(r"""([\w_][\w0-9._]*)\[(.*)""")


def current_dict(cursor_offset: int, line: str) -> Optional[LinePart]:
Expand Down