search

spoofax-shell-2017 / spoofax-shell

Apache License 2.0
0 stars 0 forks source link

Redundant import(s) #8

Open Pathemeous opened 7 years ago

Pathemeous commented 7 years ago 
    <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 commented 7 years ago

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 commented 7 years ago

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