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

[core,swift] Antlr Base Parser adapter and Swift Implementation #1698

Merged
merged 5 commits into from
Jun 15, 2019

Conversation

lsoncini
Copy link
Contributor

This pull request depends on #1658.

Before submitting a PR, please check that:

  • The PR is submitted against master. The PMD team will merge back to support branches as needed.
  • ./mvnw clean verify passes. This will build and test PMD, execute PMD and checkstyle rules. Check this for more info

PR Description:

We added an ant-file to make all Antlr4 autogenerated nodes extend AntlrBaseNode for swift, a base parser adapter for all Antlr4 autogenerated parsers and it's implementation for swift.

Team Raptor.

@ghost
Copy link

ghost commented Feb 25, 2019

1 Message
📖 No java rules are changed!

Generated by 🚫 Danger

@jsotuyod
Copy link
Member

I'd consider wrapping the exceptions in runtime exceptions that can abort the execution

@jsotuyod
Copy link
Member

jsotuyod commented Apr 1, 2019

Please, rebase this branch, as the PR with node changes is already merged.

@jsotuyod jsotuyod added the is:WIP For PRs that are not fully ready, or issues that are actively being tackled label Apr 7, 2019
@jsotuyod jsotuyod added this to the 7.0.0 milestone Apr 7, 2019
@lsoncini lsoncini mentioned this pull request Apr 13, 2019
2 tasks
@jsotuyod
Copy link
Member

@lsoncini @matifraga @tomidelucca this still needs to be rebased…

@lsoncini lsoncini force-pushed the feature/antlr-base-parser branch from 02ad0f4 to 3469834 Compare April 21, 2019 05:18
@jsotuyod jsotuyod removed the is:WIP For PRs that are not fully ready, or issues that are actively being tackled label May 24, 2019
@adangel adangel self-assigned this Jun 15, 2019
@adangel adangel added the an:enhancement An improvement on existing features / rules label Jun 15, 2019
@adangel adangel merged commit f2f0494 into pmd:pmd/7.0.x Jun 15, 2019
adangel added a commit that referenced this pull request Jun 15, 2019
@matifraga matifraga deleted the feature/antlr-base-parser branch June 15, 2019 15:19
@adangel adangel changed the title [core] [swift] Antlr Base Parser adapter and Swift Implementation [core,swift] Antlr Base Parser adapter and Swift Implementation Jan 13, 2023
@adangel adangel mentioned this pull request Jan 23, 2023
55 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
an:enhancement An improvement on existing features / rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants