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

Fix issues with play dependency #9

Merged
merged 13 commits into from
Dec 29, 2020
Merged

Fix issues with play dependency #9

merged 13 commits into from
Dec 29, 2020

Conversation

Srikrishna31
Copy link
Owner

@Srikrishna31 Srikrishna31 commented Sep 26, 2020

Work in progress to convert to a proper play application.
This repo: https://github.com/akashdayal/play-scala-rest-api-example-2.5.x-bazel/ can be used as an example to follow.

Krishna added 7 commits September 26, 2020 08:21
Move the rules_scala, google_protobuff dependencies into their own
.bzl files, and load them from WORKSPACE file, to keep it clean.

Any new dependency like this should be handled similarly.
1. Add dependency on play-routes from here: https://github.com/lucidsoftware/rules_play_routes
2. Add dependency on rules_jvm_external from here: https://github.com/bazelbuild/rules_jvm_external
3. Add dependency on play-twirl from here: https://github.com/lucidsoftware/rules_twirl
4. play-twirl is not yet working. Need to fix it in future.
Empty call to maven_install, to make sure all is well in atleast calling it.
Create the main app target, which combines model and controllers.
In progress for converting other targets.
@Srikrishna31 Srikrishna31 force-pushed the play_dependency branch 2 times, most recently from d9d5245 to 3648135 Compare December 29, 2020 12:22
@Srikrishna31 Srikrishna31 self-assigned this Dec 29, 2020
@Srikrishna31 Srikrishna31 merged commit 96f13fb into master Dec 29, 2020
@Srikrishna31 Srikrishna31 linked an issue Dec 30, 2020 that may be closed by this pull request
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.

Create full set of play dependencies
1 participant