Grammar overhaul #38
Merged
Grammar overhaul #38
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 20 commits
July 1, 2021 22:17
Also use correct token name for classdef block modifiers (#27)
|
Tests aren't running because vscode-grammar can't backtrack in the new single line block tests. Will fix tomorrow. |
|
Fixed tests. @rlivings39 @gauravsavadi You're probably busy with R2021b but what's the ETA on merging this pull |
This was referenced Jul 2, 2021
Closed
ghost
mentioned this pull request
17 tasks
This comment has been minimized.
This comment has been minimized.
gauravsavadi
added
the
CLA required
|
I have signed the CLA (I'm Munin!) however I haven't completed the work yet. Noting here that I've done readwrite operation matching via injection grammar in the |
ghost
marked this pull request as draft
ghost
marked this pull request as ready for review
|
@gauravsavadi requesting review on this PR |
gauravsavadi
added
CLA signed
ghost
mentioned this pull request
rlivings39
added a commit
to thedavidprice/language-matlab-octave
that referenced
this pull request
Jul 13, 2021
ghost
mentioned this pull request
ghost
mentioned this pull request
Gimly
pushed a commit
to Gimly/vscode-matlab
that referenced
this pull request
Aug 6, 2021
Gimly
pushed a commit
to Gimly/vscode-matlab
that referenced
this pull request
Oct 4, 2021
* Add MATLAB support library injections * Sync MATLAB grammar submodule * Validate Oniguruma patterns in injection grammars * Add missing functions (images, errors) * Update dependencies * Remove duplicated constants from injection grammar * Fix builtin grammar injection selector * Sync submodule with mathworks/MATLAB-Language-grammar#38 * Fix typo in builtin.matlab.injection * Highlight readwrite variables and properties * Add and enforce Editorconfig * Use support.function.io to avoid `storage` clashes * Enforce editorconfig in readwrite injection * Add toolboxes and simplify regexes * Undeprecate figure.Visible * Undeprecate generic properties * Fix package.json whitespace npm/npm#3180 (comment) * Add matlab.io scientific dataset packages * Fix builtin functions with identifier/dot prefixes * Fix readwrite injection selector * Prevent variables in command dualilty * Respect entities and shell highlighting * Exclude support library in shell highlighting * Fix variable injection patterns * Use two-space indent in source code * Undeprecate rm* functions * Sync with mathworks/MATLAB-Language-grammar#39 * Added extended syntax features Registers TOC, folding, workspace symbol and document symbol providers. * Register new editor providers * Re-implement Textmate tokenizer engine * Consistently use single quotes * Re-implement block folding ranges * Add tokenizer caching, fix folding provider fatal * Extract to vsctmls package * Sync editorconfig and existing whitespace * Fold variable highlighting into core grammar Refs: mathworks/MATLAB-Language-grammar#40 Co-authored-by: 4086606 <82655227+4086606@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull does numerous things:
Closes #26.
Closes #27.
Closes #35.
Closes #36.