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

[REGRESS] Simplicity of Install Instructions #4245

Merged
merged 1 commit into from Apr 21, 2015
Merged

[REGRESS] Simplicity of Install Instructions #4245

merged 1 commit into from Apr 21, 2015

Conversation

ghost
Copy link

@ghost ghost commented Apr 18, 2015

With #4204 I've started to simplify the Install instructions.

With #4231, a regression was introduced (to the freshly simplified INSTALL.md document).

Note that I had anyway not yet finished with simplifying the document fully (it's a creative process, and needs some time). I need to apply a few more things, until I can say: "Well, that's a simple installation doc"

So, it would be nice to not interfere here, and to wait until I'm finished (with this one document).

REGRESS

The text

If you don’ want to download the complete Activator distribution, there is a minimal package with no bundled dependencies. This mini-package has around 1MB and will download all the required dependencies at the first run.

is too much. This information introduces a decision to the newcomer and additionally, this information is anyway provided at the download location (or should be provided there). (Another issue is that standard/minimal distinction is not clarified).

I understand that experts have sometimes the need to share their expertise. But documentation addressed to newcomers is simply not the right place for that.

jroper added a commit that referenced this pull request Apr 21, 2015
[REGRESS] Simplicity of Install Instructions
@jroper jroper merged commit c930389 into playframework:master Apr 21, 2015
@ghost ghost deleted the patch-2 branch April 24, 2015 11:02
ClaraAllende pushed a commit to ClaraAllende/playframework that referenced this pull request Aug 28, 2015
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

1 participant