Skip to content

Commit

Permalink
Scope Data: Resolve constant.other.placeholder
Browse files Browse the repository at this point in the history
Has been discussed in an RFC and default packages PR.

See-also: sublimehq/Packages#3803 (comment)
See-also: sublimehq/Packages#3766
  • Loading branch information
FichteFoll committed Sep 11, 2023
1 parent ed8d6e9 commit de3e853
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion plugins/lib/scope_data/data.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
# https://www.sublimetext.com/docs/3/scope_naming.html
# https://manual.macromates.com/en/language_grammars#naming_conventions
#
# constant.other.{path,wildcard}: https://github.com/sublimehq/Packages/pull/3803#issuecomment-1627711666

Check failure on line 4 in plugins/lib/scope_data/data.py

View workflow job for this annotation

GitHub Actions / Flake8

E501 line too long (105 > 99 characters)
DATA = """
comment
line
Expand All @@ -24,7 +26,12 @@
boolean
null
other
placeholder
path
parent
self
wildcard
asterisk
questionmark
entity
name
Expand Down

2 comments on commit de3e853

@michaelblyons
Copy link

Choose a reason for hiding this comment

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

Why get rid of placeholder? I thought that's still important for the %s kind of stuff.

@FichteFoll
Copy link
Member Author

Choose a reason for hiding this comment

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

True, thanks for the heads-up.

Please sign in to comment.