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

Add lexer for Smithy #1878

Closed
immanuelqrw opened this issue Aug 5, 2021 · 0 comments · Fixed by #1879
Closed

Add lexer for Smithy #1878

immanuelqrw opened this issue Aug 5, 2021 · 0 comments · Fixed by #1879
Milestone

Comments

@immanuelqrw
Copy link
Contributor

AWS has a new interface definition language called Smithy.

Smithy already has a lexer using pygments' classes, so this only needs to be modified to be local and then merged in.

immanuelqrw added a commit to immanuelqrw/pygments that referenced this issue Aug 5, 2021
* Added Smithy Lexer
* Added Smithy Lexer auhtor
* Documented Smithy as a supported language
* Added Smithy test file and output
Anteru pushed a commit that referenced this issue Aug 8, 2021
* Added Smithy Lexer (#1878)

* Added Smithy Lexer
* Added Smithy Lexer auhtor
* Documented Smithy as a supported language
* Added Smithy test file and output

* Updated Smithy Lexer

* Added Standard file heading with copyright and license
* Used `words` method for optimization, instead of bare regex
* Specified whitespace punctuation in root
* Updated aliases to only contain lowercase names to pass `test_basic_api` tests
* Updated regexes lightly to fit regexlint rules (removing duplicate characters in group `-`)
* Fixed regexes with errors in regexlint rules (Escaping brackets, gaps in capture groups)

* Ran mapping script to fix build check

* Ran mapping to update after changing aliases in previous commit
@Anteru Anteru linked a pull request Aug 8, 2021 that will close this issue
@Anteru Anteru added this to the 2.10 milestone Aug 8, 2021
@Anteru Anteru closed this as completed Aug 8, 2021
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 a pull request may close this issue.

2 participants