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 Basic Ada Language Support #1021

Closed
wants to merge 2 commits into from

Conversation

btmalone
Copy link

Added Ada defined keywords and types, as well as support for
strings, integers, operators, and comments. Added both body
and spec file extensions, .adb and .ads respectively, as well
as AdaCore GPRbuild tool file extension, .gpr.

Resolves #824

Added Ada defined keywords and types, as well as support for
strings, integers, operators, and comments. Added both body
and spec file extensions, .adb and .ads respectively, as well
as AdaCore GPRbuild tool file extension, .gpr.

Resolves rouge-ruby#824
Copy link
Contributor

@vidarh vidarh left a comment

Choose a reason for hiding this comment

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

Hi, thanks for submitting this. I'm trying to help triage pull requests to see if we can get the backlog down... Only comment I have for this one is your sample. Thoughts? The lexer itself seems fine as a first pass.

@@ -0,0 +1,343 @@
------------------------------------------------------------------------------
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure it's appropriate to use a GPL'd file as a sample for Rouge. Rouge is MIT licensed, and while it may not be a problem to include it as a sample, it might be preferable to use another sample. Also, the point of the sample file is to show the effect of the different rules of the lexer. Since this lexer is very basic, I don't think it's necessary to have such a large sample.

Remove the GPL'd sample file, and replace it with a simpler sample
file that shows all the types, keywords and symbols.
@pyrmont
Copy link
Contributor

pyrmont commented May 29, 2019

@btmalone I'm sorry it's taken so long to get to this. The updated sample is definitely simpler but perhaps a little too simple. Could you expand it a little to include some procedures as well?

@pyrmont pyrmont added the author-action The PR has been reviewed but action by the author is needed label May 29, 2019
@pyrmont
Copy link
Contributor

pyrmont commented Jul 11, 2019

This PR has been superseded by #1255. Thanks for the submission nevertheless, @btmalone!

@pyrmont pyrmont closed this Jul 11, 2019
@pyrmont pyrmont removed the author-action The PR has been reviewed but action by the author is needed label Jul 11, 2019
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.

Ada language support
3 participants