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

Getting started links #62

Closed
wants to merge 4 commits into from
Closed

Getting started links #62

wants to merge 4 commits into from

Conversation

zwalchuk
Copy link

@zwalchuk zwalchuk commented Feb 5, 2018

Update getting started links for Odroid C1 and XU4

Connects to #44

@resin-jenkins
Copy link

Can one of the admins verify this patch?

@zwalchuk zwalchuk requested a review from floion February 5, 2018 16:52
@resin-jenkins
Copy link

Can one of the admins verify this patch?

CHANGELOG.md Outdated
@@ -6,6 +6,7 @@ Change log

* Update the resin-yocto-scripts submodule to 6f7a9ab326bb822196e3e48b08ef1d73d08caec6 (on master branch) [Florin]
* Use the new offsets for u-boot env and Trusted Zone binary blob that our new u-boot (version 2017.09) expects [Florin]
* Update getting starting links [Zach]
Copy link
Contributor

Choose a reason for hiding this comment

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

@zwalchuk this needs to sit on top; new additions to changelog are added on top

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, please change the commit log to be like:

CHANGELOG: Update getting started links

linux: 'http://docs.resin.io/#/pages/installing/gettingStarted-ODROID-C1.md#on-mac-and-linux'
windows: 'https://docs.resin.io/installing/gettingStarted-Odroid-C1/#creating-your-first-application'
osx: 'https://docs.resin.io/installing/gettingStarted-Odroid-C1/#creating-your-first-application'
linux: 'https://docs.resin.io/installing/gettingStarted-Odroid-C1/#creating-your-first-application'
Copy link
Contributor

Choose a reason for hiding this comment

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

Commit log should be instead like:

odroid-c1.coffee: Update getting started links

Make sure getting started links match the appropriate getting started guides.

Copy link
Author

Choose a reason for hiding this comment

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

Fixed

linux: 'http://docs.resin.io/#/pages/installing/gettingStarted-ODROID-XU4.md#on-mac-and-linux'
windows: 'https://docs.resin.io/installing/gettingStarted-Odroid-C1/#creating-your-first-application'
osx: 'https://docs.resin.io/installing/gettingStarted-Odroid-C1/#creating-your-first-application'
linux: 'https://docs.resin.io/installing/gettingStarted-Odroid-C1/#creating-your-first-application'
Copy link
Contributor

Choose a reason for hiding this comment

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

The commit log needs to be like the following:

odroid-xu4.coffee: Update XU4 getting started links

Copy link
Author

Choose a reason for hiding this comment

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

Fixed

@floion
Copy link
Contributor

floion commented Feb 6, 2018

This PR also needs rebasing

@zwalchuk zwalchuk requested a review from floion February 6, 2018 20:44
@floion
Copy link
Contributor

floion commented Feb 7, 2018

@resin-jenkins add to whitelist

@floion
Copy link
Contributor

floion commented Feb 7, 2018

@zwalchuk can you rebase this? There seems to be an issue with how you rebased last time

@zwalchuk
Copy link
Author

zwalchuk commented Feb 9, 2018

@floion These links are all going to be changing again today or early next week, I'm going to close this PR and start over doing it correctly when the new links are live

@zwalchuk zwalchuk closed this Feb 9, 2018
@zwalchuk zwalchuk deleted the getting-started-links branch February 9, 2018 16:33
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