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

Add server instructions to README #1931

Merged

Conversation

kohrVid
Copy link
Contributor

@kohrVid kohrVid commented Dec 29, 2017

Mailing List thread:

I thought it would be worth adding information on how to run the server to the README because the instructions I found in many of the tutorials and on the official website are out-of-date. I stumbled upon the above post in the mailing list by chance which contains the correct command for Lift v.3.

@karma4u101
Copy link
Member

👍 Awesome LGTM. Some more eyes on this and it will be merged, thanks for the update.

Copy link
Member

@farmdawgnation farmdawgnation left a comment

Choose a reason for hiding this comment

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

Thanks for this contribution and welcome to Lift!

The expectation when I reorganized this last was that folks would click through to the xsbt-web-plugin documentation and find instructions there. However, I can see that didn't quite go as I planned, so I think the right thing to do here is add some explicit guidance.

My one issue here is that the way things are ordered is a bit misleading. The instructions you've provided only apply to people using sbt instructions, but it appears below the maven instructions.

What do you think about creating two copies of this text at the bottom of each SBT section?

Bonus points if you're able to locate some corresponding instructions for starting the server from maven and add those as well.

Have added instructions on how to run the server in the latest version
of Lift to the README.
@kohrVid kohrVid force-pushed the add-server-instructions-to-readme branch from 396c837 to 9362cd9 Compare January 3, 2018 02:17
@kohrVid
Copy link
Contributor Author

kohrVid commented Jan 3, 2018

Thank you both for the feedback. @farmdawgnation, I've moved the server instructions to the two SBT sections of the README like you suggested but I'm not very familiar with Maven and so was unable to find instructions for running the server that way. I hope that's ok.
Let me know if you'd like me to change anything else

Copy link
Member

@farmdawgnation farmdawgnation left a comment

Choose a reason for hiding this comment

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

@kohrVid I made a minor change to the header levels to avoid any possible confusion that the server sections "belong" to the sbt sections. other than that, this looks golden.

Thank you for this contribution!

@farmdawgnation farmdawgnation merged commit 21eba93 into lift:master Jan 3, 2018
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.

3 participants