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

Divide targets in different directories #120

Merged
merged 12 commits into from
Jan 20, 2021

Conversation

ladislas
Copy link
Member

No description provided.

@ladislas ladislas force-pushed the feature/ladislas-divide-leka-targets branch 6 times, most recently from d82d349 to 6f877db Compare January 17, 2021 21:03
@ladislas ladislas force-pushed the feature/ladislas-divide-leka-targets branch 4 times, most recently from db7bf4e to 06057f4 Compare January 17, 2021 21:48
For target LEKA_V1_2_DEV and files:

- .bin
- .hex
- .elf
@ladislas ladislas force-pushed the feature/ladislas-divide-leka-targets branch from 06057f4 to da6ba93 Compare January 17, 2021 22:03
Copy link
Member

@YannLocatelli YannLocatelli left a comment

Choose a reason for hiding this comment

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

Il y a deux erreurs de compilation mineurs (code_analysis et configuration LEKA_DISCO) et une proposition d'amélioration (custom_targets.json), sinon tout fonctionne bien!

Makefile Outdated Show resolved Hide resolved
targets/custom_targets.json Show resolved Hide resolved
mbed_app.json Show resolved Hide resolved
mbed_app.json Show resolved Hide resolved
@codeclimate
Copy link

codeclimate bot commented Jan 18, 2021

Code Climate has analyzed commit 3e3e436 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 100.0% (0.0% change).

View more on Code Climate.

@ladislas ladislas merged commit 57e3e2e into develop Jan 20, 2021
@ladislas ladislas deleted the feature/ladislas-divide-leka-targets branch January 27, 2021 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
01 - type: task Something to do
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants