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

GHC Core #1377

Merged
merged 19 commits into from Dec 23, 2019
Merged

GHC Core #1377

merged 19 commits into from Dec 23, 2019

Conversation

supersven
Copy link
Contributor

Add Lexer for GHC Core

Core is an intermediate language of the Glasgow Haskell Compiler.

@pyrmont pyrmont self-assigned this Dec 15, 2019
@pyrmont pyrmont added the needs-review The PR needs to be reviewed label Dec 15, 2019
Copy link
Contributor

@pyrmont pyrmont left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like a really impressive lexer! Thanks for submitting the PR. I've left some comments above. Let me know if anything is unclear :)

In addition, could you cut down the size of the visual sample? It's too long at the moment. Ideally, it should be just enough to visually test the rules.

lib/rouge/demos/ghc-core Outdated Show resolved Hide resolved
lib/rouge/lexers/ghc_core.rb Outdated Show resolved Hide resolved
lib/rouge/lexers/ghc_core.rb Outdated Show resolved Hide resolved
@pyrmont pyrmont added author-action The PR has been reviewed but action by the author is needed and removed needs-review The PR needs to be reviewed labels Dec 18, 2019
@pyrmont
Copy link
Contributor

pyrmont commented Dec 19, 2019

@supersven Sorry, one minor change: I renamed the class to be a little more consistent with the general approach taken with Rouge's lexers. It's not always universally followed but it should be done this way.

@supersven
Copy link
Contributor Author

Hey @pyrmont ,

Thanks a lot for the review and your changes!

I forgot to mention one thing: If you like, you can squash the commits during merge. I think that my "baby step" commits might be more confusing than helping in the long run ... 😄

@pyrmont pyrmont merged commit 78bc930 into rouge-ruby:master Dec 23, 2019
@pyrmont
Copy link
Contributor

pyrmont commented Dec 23, 2019

@supersven Yep, I always squash the PR commits into a single commit before merging. Helps to keep the commit history nice and clean :) Thanks again for all the work! 🎉

The next version of Rouge will be out on Tuesday 14 January. This will be part of that release :)

@pyrmont pyrmont removed the author-action The PR has been reviewed but action by the author is needed label Dec 23, 2019
@supersven
Copy link
Contributor Author

@pyrmont ,
Fantastic! Thanks a lot for the review! 👍

@supersven supersven deleted the ghc-core branch December 30, 2019 09:18
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