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

Redundant import(s) #8

Open
Pathemeous opened this issue Jun 16, 2017 · 2 comments
Open

Redundant import(s) #8

Pathemeous opened this issue Jun 16, 2017 · 2 comments

Comments

@Pathemeous
Copy link
Member

    <dependency>
      <groupId>org.metaborg</groupId>
      <artifactId>org.metaborg.meta.lang.dynsem.interpreter</artifactId>
      <version>${metaborg-version}</version>
    </dependency>

    <dependency>
      <groupId>org.metaborg</groupId>
      <artifactId>org.metaborg.meta.interpreter.framework</artifactId>
      <version>${metaborg-version}</version>
    </dependency>

The second import seems redundant (commenting doesn't fail the console build)

@justinvdk
Copy link

without doing a mvn install maven complains about missing dependencies. The dynsem.interpreter could be removed now with stratego I think, but I'm not sure about the second one.

@Pathemeous
Copy link
Member Author

@hendrikvanantwerpen you added these. Do you know in what way they are relevant?

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

No branches or pull requests

2 participants