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

Paths #1

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Paths #1

wants to merge 5 commits into from

Conversation

BabylonZeus
Copy link

No description provided.

@ubourdon
Copy link
Member

Yo
Ta pull request me semble un peu compliquée ;)
Du coup ça m'a poussé à commencer à de-stubber le code.
Je l'ai déjà fait pour l'url GET /project/*projectName

Me reste à unstubber les path vers scalastyle & scct.
Pour que cela marche et reste simple dans un premier temps il faudra :
- installer play/sbt(/maven/graddle/...)
- installer le plugin scct dans play/sbt(/maven/graddle/...) au préalable
- installer scalastyle-batch
- déclarer les paths de play/sbt(/maven/graddle/...) et de scalastyle-batch.jar dans un fichier de properties

Dans un second temps, nous pourrons mettre en place un service qui installe lui même les dépendances nécessaires au fonctionnement de scala-radar (en packageant par exemple tous ces soft dans l'install de scala-radar - un zip)

Du coup je laisse en attente cette pull request pour l'instant. Tout en essayant de répondre aux problèmes soulevés par l'issue le plus rapidement possible.
Ok pour toi ?

@BabylonZeus
Copy link
Author

J'ai pas tout en tête de ce qui est porté en l'état, ce que tu proposes me semble raisonnable, sans pour autant faire la correspondance avec le code existant pour l'instant. Ok pour ne pas prendre en compte la PR.

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.

2 participants