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

boards/nucleo64*: rename boards to use their ST marketing name #8649

Merged
merged 17 commits into from
May 23, 2018

Conversation

aabadie
Copy link
Contributor

@aabadie aabadie commented Feb 27, 2018

Contribution description

This PR renames all nucleo-64 boards to make them use the marketing name used in ST documentation, see http://www.st.com/en/evaluation-tools/stm32-mcu-nucleo.html

For the record, I used a semi-automatic renaming strategy with the following commands:

git mv boards/nucleo-l476 boards/nucleo-l476rg
git grep -l 'nucleo-l476' | xargs sed -i 's/nucleo-l476/nucleo-l476rg/g'
git grep -l 'Nucleo-L476' | xargs sed -i 's/Nucleo-L476/Nucleo-L476RG/g'
git add boards/nucleo-l476rg
git commit -m 'boards/nucleo-l476rg: rename to standard st name'
git add examples
git commit -m 'examples: use new nucleo-l476rg name'
git add tests
git commit -m 'tests: use new nucleo-l476rg name'

And I have a simple bash script that generates the commands.

Issues/PRs references

Partially address #8096

@aabadie aabadie added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: boards Area: Board ports labels Feb 27, 2018
@aabadie aabadie added Community: Hack'n'ACK candidate This PR is a candidate for review and discussion during one of RIOT's monthly Hack'n'ACK parties and removed Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation labels Feb 27, 2018
@kaspar030
Copy link
Contributor

I think one commit per board is enough...

@aabadie aabadie force-pushed the pr/nucleo-rename branch 2 times, most recently from 6248523 to 386f2a5 Compare March 13, 2018 09:01
@aabadie
Copy link
Contributor Author

aabadie commented Mar 13, 2018

I think one commit per board is enough...

done

@aabadie
Copy link
Contributor Author

aabadie commented May 22, 2018

@vincent-d, rebased to fix conflicts.

Copy link
Member

@vincent-d vincent-d left a comment

Choose a reason for hiding this comment

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

ACK

@vincent-d
Copy link
Member

And go!

@vincent-d vincent-d merged commit 93ebf38 into RIOT-OS:master May 23, 2018
@aabadie aabadie deleted the pr/nucleo-rename branch June 1, 2018 18:58
@cladmi cladmi added this to the Release 2018.07 milestone Jul 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: boards Area: Board ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Community: Hack'n'ACK candidate This PR is a candidate for review and discussion during one of RIOT's monthly Hack'n'ACK parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants