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 official link on Heroku Deployment #16719

Closed
wants to merge 1 commit into from
Closed

Add official link on Heroku Deployment #16719

wants to merge 1 commit into from

Conversation

Tetsuya3850
Copy link
Contributor

The current doc’s instructions on how to deploy spring boot apps to Heroku is not kind enough.
https://docs.spring.io/spring-boot/docs/2.1.4.RELEASE/reference/htmlsingle/#cloud-deployment-heroku

“The most common deployment workflow for Heroku deployments is to git push the code to production, as shown in the following example: Your application should now be up and running on Heroku.”

Heroku’s official guide on how to do so is very thorough.
https://devcenter.heroku.com/articles/deploying-spring-boot-apps-to-heroku

Therefore, it will be nice if a link to it can be added.

P.S. I think that the snippet of git push heroku master does not provide much information and is just taking up space.

The current doc’s instructions on how to deploy spring boot apps to Heroku is not kind enough.
https://docs.spring.io/spring-boot/docs/2.1.4.RELEASE/reference/htmlsingle/#cloud-deployment-heroku

“The most common deployment workflow for Heroku deployments is to git push the code to production, as shown in the following example: Your application should now be up and running on Heroku.”  

Heroku’s official guide on how to do so is very thorough.
https://devcenter.heroku.com/articles/deploying-spring-boot-apps-to-heroku

Therefore, it will be nice if a link to it can be added.

P.S. I think that the snippet of git push heroku master does not provide much information and is just taking up space.
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label May 4, 2019
@philwebb philwebb added type: documentation A documentation update and removed status: waiting-for-triage An issue we've not yet triaged labels May 6, 2019
@philwebb philwebb added this to the 2.1.x milestone May 6, 2019
@philwebb
Copy link
Member

philwebb commented May 6, 2019

Thanks! Linking to the official docs is great idea.

snicoll pushed a commit that referenced this pull request May 6, 2019
snicoll added a commit that referenced this pull request May 6, 2019
* pr/16719:
  Polish "Add link to Heroku article"
  Add link to Heroku article
@snicoll snicoll self-assigned this May 6, 2019
@snicoll snicoll closed this in d8f1bd3 May 6, 2019
@snicoll snicoll modified the milestones: 2.1.x, 2.1.5 May 6, 2019
@snicoll
Copy link
Member

snicoll commented May 6, 2019

@Tetsuya3850 thanks for the PR. I've polished it in d8f1bd3 as the link in your PR was in markdown format (and wouldn't have rendered properly).

@Tetsuya3850
Copy link
Contributor Author

Ah! I wasn't aware of that format requirement. Thank you for the polish.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation A documentation update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants