Skip to content
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.

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

[ST4] Some changes to build system completions & updating it's syntax. #320

Merged
merged 4 commits into from Jan 6, 2021

Conversation

Ultra-Instinct-05
Copy link
Contributor

@Ultra-Instinct-05 Ultra-Instinct-05 commented Dec 19, 2020

This PR

  1. Adds completions for kill_previous (introduced in 4092 to kill previously running builds) & update_annotations_only (introduced in 4050) to the build system completions file. These are also added to the syntax file for appropriate highlighting.
  2. It removes any phantom related exec args from the completions & syntax file since phantoms are no longer used to show build errors in the buffer from 4050 onwards.

This commit adds the `kill_previous` arg to the completions file(s).
It was introduced in Build 4092 and meant to kill any previously running
builds before starting a new one.
This commit removes completions for phantom related exec args since
annotations have now replaced them, so they no longer have any meaning
in the new 4xxx builds starting from Build 4050.
This commit
1. Removes the phantoms related exec args for reasons already stated in
   the previous commit.
2. Adds the `kill_previous` arg for highlighting purposes.
This commit adds the `update_annotations_only` arg to the completions file(s).
It was introduced in Build 4050. There seems to have been a corresponding
`hide_annotations_only` arg as well but that looks to have been removed
in Build 4067 for some reason.
@FichteFoll
Copy link
Member

I'm wondering whether we should highlight these very specific args in build systems in the first place, since they are primarily used for other invocations of the exec command through key bindings or the command palette rather than a build system, but I'll decide on that later.

Thanks.

@FichteFoll FichteFoll merged commit 9c81856 into SublimeText:master Jan 6, 2021
@FichteFoll FichteFoll added this to the 3.3.0 milestone Jan 6, 2021
@Ultra-Instinct-05 Ultra-Instinct-05 deleted the update_build branch January 7, 2021 04:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants