Skip to content

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

Merged
adangel merged 5 commits into
pmd:pmd/7.0.xfrom
teamraptor:feature/antlr-base-parser
Jun 15, 2019
Merged

[core,swift] Antlr Base Parser adapter and Swift Implementation#1698
adangel merged 5 commits into
pmd:pmd/7.0.xfrom
teamraptor:feature/antlr-base-parser

Conversation

@lsoncini

Copy link
Copy Markdown
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

ghost commented Feb 25, 2019

Copy link
Copy Markdown
1 Message
📖 No java rules are changed!

Generated by 🚫 Danger

@jsotuyod

Copy link
Copy Markdown
Member

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

@jsotuyod

jsotuyod commented Apr 1, 2019

Copy link
Copy Markdown
Member

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
Copy Markdown
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
Comment thread pmd-core/src/main/java/net/sourceforge/pmd/lang/antlr/AntlrBaseParser.java Outdated
Comment thread pmd-swift/src/main/java/net/sourceforge/pmd/lang/swift/SwiftParserAdapter.java Outdated
@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