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 #122: Add autocomplete bar #198

Merged
merged 3 commits into from Jul 20, 2016
Merged

Fix #122: Add autocomplete bar #198

merged 3 commits into from Jul 20, 2016

Conversation

jvican
Copy link
Contributor

@jvican jvican commented Jul 19, 2016

No description provided.

@MasseGuillaume
Copy link
Contributor

MasseGuillaume commented Jul 19, 2016

cool the failing test is because of the sbt project rename. I suggest we keep the name server

.settings(name := "server")

@jvican
Copy link
Contributor Author

jvican commented Jul 19, 2016

Unfortunately, that doesn't work. I tried it out and sbt doesn't respond to the name change. The culprit of this behaviour seems to be the SBT scalajs plugin, which uses a macro to set the project names.

* Bring back Scalajs!

* Add autocompletion and JS dependencies.
@jvican
Copy link
Contributor Author

jvican commented Jul 20, 2016

I've found another way to achieve the same. This PR is ready for review & merge (I have rebased).

)
)

lazy val client = scaladex.js.dependsOn(model)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dependsOn(model) I'm not sure this still holds

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's what you put, so we better leave it.

SBT detects now client and server as independent projects with their own
names. Also, when compiling the server for the first time the client is
also compiled as a dependency.
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.

None yet

2 participants