mbed_cli_2: Add migration guide #1441
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Patater force-pushed the add-mbed-cli-2-migration-guide branch 5 times, most recently from f59b4f9 to 841ba5a Compare 
rwalton-arm requested changes Apr 7, 2021
5 tasks
wernerlewis suggested changes Apr 7, 2021
| Could add "Fixes #227" in the PR description to link to the issue |
Added |
Ah I didn't realise it was a different project. It seems GitHub may not pick up links sometimes. |
Patater force-pushed the add-mbed-cli-2-migration-guide branch from 841ba5a to 7b7dc0b Compare wernerlewis suggested changes Apr 9, 2021
rwalton-arm requested changes Apr 9, 2021
Patater force-pushed the add-mbed-cli-2-migration-guide branch from 7b7dc0b to 023a2be Compare 
rwalton-arm approved these changes Apr 9, 2021
LDong-Arm reviewed Apr 9, 2021
| ^ A couple of last-minute comments |
Patater force-pushed the add-mbed-cli-2-migration-guide branch from 023a2be to 793e32e Compare 
rwalton-arm approved these changes Apr 13, 2021
wernerlewis approved these changes Apr 14, 2021
Patater force-pushed the add-mbed-cli-2-migration-guide branch from 793e32e to 375ffd7 Compare 
Add a migration guide to help show how to do things that Mbed CLI 1 could do using Mbed CLI 2. Fixes ARMmbed/mbed-tools#227
Add missing step to generate an `mbed_config.cmake` file. Without this, it wasn't possible to follow the instructions given. Update the CMake build directory to match the path naming convention that Mbed CLI 2 uses, so that manually moving around the `configure` subcommand generated `mbed_config.cmake` isn't necessary. Matching path conventions isn't required and the path can be any valid path that contains a valid `mbed_config.cmake` file. Fixes ARMmbed/mbed-tools#263 Fixes ARMmbed/mbed-tools#260
Patater force-pushed the add-mbed-cli-2-migration-guide branch from 375ffd7 to 3fd85ee Compare 
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
Add a migration guide to help show how to do things that Mbed CLI 1
could do using Mbed CLI 2.
Fixes ARMmbed/mbed-tools#227