Skip to content

refactor(apex): rearrange class for readability#29

Merged
dschach merged 4 commits intomainfrom
refactor/rearrange
Nov 6, 2023
Merged

refactor(apex): rearrange class for readability#29
dschach merged 4 commits intomainfrom
refactor/rearrange

Conversation

@dschach
Copy link
Copy Markdown
Collaborator

@dschach dschach commented Nov 6, 2023

refactor(params): simplify parameter usage, moving most to top-level
Of note: moving keyword lists to a separate file prevents the build/test process

refactor: moved some matchers to named objects for better readability

test: add illegal strings to prevent recognizing as other languages (clojure, gams, mathematica)

test: update expected markup after removing extra scope words

BEGIN_COMMIT_OVERRIDE
refactor(apex): rearrange class for readability

refactor(params): simplify parameter usage, moving most to top-level
refactor: moved some matchers to named objects for better readability
test: add illegal strings to prevent recognizing as other languages (clojure, gams, mathematica)
test: update expected markup after removing extra scope words
END_COMMIT_OVERRIDE

@dschach dschach changed the title refactor: rearrange class for readability refactor(apex): rearrange class for readability Nov 6, 2023
@dschach dschach merged commit e73a679 into main Nov 6, 2023
dschach pushed a commit that referenced this pull request Nov 7, 2023
🤖 I have created a release *beep* *boop*
---


##
[1.3.2](v1.3.1...v1.3.2)
(2023-11-07)


### Code Refactoring

* **apex:** rearrange class for readability
([#29](#29))
([e73a679](e73a679))
* rearrange modules to better order for parsing
([#30](#30))
([b8e8a6f](b8e8a6f))


### Styles

* **monokai-sublime-apex:** update theme subscopes
([#27](#27))
([837b79f](837b79f))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

1 participant