Skip to content

recommend sbt, not Activator #530

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

Merged
merged 2 commits into from
Nov 16, 2016
Merged

Conversation

SethTisue
Copy link
Member

and remove all mention of Activator from the site
(well, except from some old release announcements)

and remove all mention of Activator from the site
(well, except from some old release announcements)
@SethTisue
Copy link
Member Author

SethTisue commented Nov 15, 2016

@SethTisue
Copy link
Member Author

screen shot 2016-11-15 at 3 32 32 pm

@tpolecat
Copy link

tpolecat commented Nov 16, 2016

I think I might prefer landing at http://www.scala-sbt.org/ since it's more attractive and it links to a nicer download page that you would otherwise miss. It would also sidestep the problem of the link being version-specific. But in any case this is a step in the direction I'd like to go so 👍

@SethTisue
Copy link
Member Author

I think I might prefer landing at http://www.scala-sbt.org/ since it's more attractive and it links to a nicer download page that you would otherwise miss

yeah, I was unsure about that. Eugene, Dale, you want to make the call there?

@eed3si9n
Copy link
Member

Sure. Landing on the landing page makes sense.

(in response to reviewer feedback)
</a>
<p class="install"><a style="margin-left: 124px;" href="http://www.lightbend.com/activator/download">Need help installing?</a></p>
</div>
sbt is a popular build tool for Scala projects.<br/>
Copy link
Member

Choose a reason for hiding this comment

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

Should sbt be capitalized here? It is the beginning of a sentence.

Copy link
Member

Choose a reason for hiding this comment

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

sbt is a proper noun like iPhone and eBay.

@@ -7,7 +7,6 @@ title: Documentation Contributions
There are several ways you can help out with the improvement of Scala documentation. These include:

* API Documentation in Scaladoc
* Code examples and tutorials in activator templates.
Copy link
Member

Choose a reason for hiding this comment

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

Sbt 13.13 has a new command that provides templates. Maybe this is worth mentioning.

Choose a reason for hiding this comment

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

Would it be possible for sbt to provide some examples for basic projects?

Copy link
Member

Choose a reason for hiding this comment

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

I agree that it would be good to link people to either documentation about new or some kind of template. @dwijnand and/or @eed3si9n, do you have any suggestions here?

Copy link
Member

Choose a reason for hiding this comment

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

Anyway, this is the wrong place to suggest these changes. I think we're making suggestions for improving the getting started page, which is not the file we're commenting on (we're commenting on a contributing page)

Copy link
Member

Choose a reason for hiding this comment

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

For the getting started page, you can point to http://www.scala-sbt.org/0.13/docs/sbt-new-and-Templates.html

Copy link
Member

@heathermiller heathermiller left a comment

Choose a reason for hiding this comment

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

LGTM. Though I'd like to point out that the comments pointing out the sbt new command would be great for a rethink of the getting started guide. Perhaps also using some tips from @djspiewak's getting started guide gist

@@ -7,7 +7,6 @@ title: Documentation Contributions
There are several ways you can help out with the improvement of Scala documentation. These include:

* API Documentation in Scaladoc
* Code examples and tutorials in activator templates.
Copy link
Member

Choose a reason for hiding this comment

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

I agree that it would be good to link people to either documentation about new or some kind of template. @dwijnand and/or @eed3si9n, do you have any suggestions here?

@@ -7,7 +7,6 @@ title: Documentation Contributions
There are several ways you can help out with the improvement of Scala documentation. These include:

* API Documentation in Scaladoc
* Code examples and tutorials in activator templates.
Copy link
Member

Choose a reason for hiding this comment

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

Anyway, this is the wrong place to suggest these changes. I think we're making suggestions for improving the getting started page, which is not the file we're commenting on (we're commenting on a contributing page)

@SethTisue
Copy link
Member Author

I'm going to go ahead and merge this, and leave it to someone else to add stuff about sbt new if they like, in a separate PR. (I think it's a good idea.) Thanks all for the review. We can always go on and make further changes.

@SethTisue SethTisue merged commit 3b80809 into scala:master Nov 16, 2016
@SethTisue SethTisue deleted the sbt-not-activator branch November 16, 2016 16:47
@SethTisue
Copy link
Member Author

on the subject of possible revisions to the getting-started page, I opened #536 (but I probably won't tackle it myself)

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.

7 participants