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

Improve formatting and update to be compatible with changed plug-in API #115

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

icaven
Copy link

@icaven icaven commented Aug 22, 2024

Addresses the items described in issue #114.

Also incorporates the changes in pull requests #109 and #112, and updates the org.jetbrains.intellij version to 1.17 as identified in PR #107.

icaven added 8 commits May 16, 2024 11:10
Comment out keywords in lexer that are overridden in BOSL2.
Update the tests.
…ous issue).

Add an element in the parser to abstract assert and echo calls.

Flag as an error repeated commas.
Update the tests.
…, scale, etc. in the lexer.

This will allow these keywords to be highlighted.  These are now parsed as overridable keywords.

Modify the indentation of non-block statements.  This means that child operations are not indented unless surrounded by braces.

Update the tests.
Wrap long module and function call parameter lists at a parameter.
Fix customizer highlighting for numbers.
Prevent customizer comments from being wrapped.
Update the test result files.
Update gradle build settings.
Specify ActionUpdateThread type.
Remove some deprecated function calls.
@mjparme
Copy link

mjparme commented Dec 31, 2024

The original developer seems to have abandoned the project. Looks like there are some PRs with some good changes in them. Wonder if maybe someone wants to merge them in their fork and then release the plugin from that fork?

I pulled in the build changes from this one so I could at least build it.

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.

2 participants