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

Integrate metrix simulator #3

Merged
merged 58 commits into from
Mar 19, 2021
Merged

Integrate metrix simulator #3

merged 58 commits into from
Mar 19, 2021

Conversation

lecourtoisflo
Copy link
Contributor

@lecourtoisflo lecourtoisflo commented Feb 15, 2021

Please check if the PR fulfills these requirements (please use '[x]' to check the checkboxes, or submit the PR and then click the checkboxes)

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

@lecourtoisflo lecourtoisflo added the enhancement New feature or request label Feb 15, 2021
@lecourtoisflo lecourtoisflo force-pushed the metrix-cpp branch 3 times, most recently from 975383c to befdfb3 Compare February 16, 2021 07:58
README.md Show resolved Hide resolved
@lecourtoisflo lecourtoisflo force-pushed the metrix-cpp branch 3 times, most recently from 5df761a to bb82f0c Compare February 24, 2021 09:27
Le Courtois Florent added 21 commits March 8, 2021 08:48
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
CI produces a result equivalent to local result but different in
metrixOut file

Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Le Courtois Florent added 2 commits March 8, 2021 11:10
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Le Courtois Florent added 4 commits March 8, 2021 16:24
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Le Courtois Florent added 14 commits March 9, 2021 11:37
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
Signed-off-by: Le Courtois Florent <[email protected]>
.github/workflows/ci.yml Show resolved Hide resolved
.github/workflows/qa_pr_cpp.yml Outdated Show resolved Hide resolved
.github/workflows/qa_pr_cpp.yml Show resolved Hide resolved
README.md Outdated
```
###### Checkstyle
This project uses [clang-tidy](https://clang.llvm.org/extra/clang-tidy/) to verify the code style. This tool is provided with the clang extra tools. To enable the code style checking, add the `-DCMAKE_CXX_CLANG_TIDY=clang-tidy` flag to the configure command.

Choose a reason for hiding this comment

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

You should mention that you provide also a clang-format configuration file to help developers to follow the coding style

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK

metrix-simulator/CMakeLists.txt Show resolved Hide resolved
metrix-simulator/CMakeLists.txt Show resolved Hide resolved
metrix-simulator/etc/METRIX_En.dic Show resolved Hide resolved
Mathieu BAGUE and others added 2 commits March 15, 2021 12:00
@sonarcloud
Copy link

sonarcloud bot commented Mar 16, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@sonarcloud
Copy link

sonarcloud bot commented Mar 16, 2021

@mathbagu mathbagu merged commit 1f971a2 into main Mar 19, 2021
@mathbagu mathbagu deleted the metrix-cpp branch March 19, 2021 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants