Skip to content
This repository has been archived by the owner on Jun 10, 2021. It is now read-only.

Fix uncrustify linting #89

Merged
merged 1 commit into from Feb 4, 2020
Merged

Conversation

dabonnie
Copy link
Contributor

@dabonnie dabonnie commented Feb 4, 2020

Signed-off-by: Devin Bonnie dbbonnie@amazon.com

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>
@codecov
Copy link

codecov bot commented Feb 4, 2020

Codecov Report

Merging #89 into master will increase coverage by 0.04%.
The diff coverage is 29.62%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
+ Coverage   39.44%   39.49%   +0.04%     
==========================================
  Files          33       33              
  Lines        1374     1375       +1     
  Branches      822      822              
==========================================
+ Hits          542      543       +1     
  Misses         65       65              
  Partials      767      767
Flag Coverage Δ
#unittests 39.49% <29.62%> (+0.04%) ⬆️
Impacted Files Coverage Δ
...r/test/system_metrics_collector/test_collector.cpp 48.78% <0%> (ø) ⬆️
...rage_statistics/test_moving_average_statistics.cpp 59.22% <0%> (ø) ⬆️
...llector/src/system_metrics_collector/utilities.cpp 40.21% <0%> (ø) ⬆️
...tor/test_linux_process_memory_measurement_node.cpp 42.85% <0%> (ø) ⬆️
...etrics_collector/test_linux_memory_measurement.cpp 28.77% <0%> (ø) ⬆️
...trics_collector/test_periodic_measurement_node.cpp 25.8% <0%> (ø) ⬆️
...test/system_metrics_collector/test_composition.cpp 34.61% <100%> (ø) ⬆️
...em_metrics_collector/periodic_measurement_node.cpp 50.54% <50%> (ø) ⬆️
...r/test/system_metrics_collector/test_utilities.cpp 21.5% <50%> (+0.85%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update beda3ad...5a164ef. Read the comment docs.

@dabonnie dabonnie merged commit cce237e into master Feb 4, 2020
@dabonnie dabonnie deleted the dabonnie/fix-uncrustify-linting branch February 4, 2020 22:41
dabonnie added a commit that referenced this pull request Feb 13, 2020
Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>
dabonnie added a commit that referenced this pull request Feb 15, 2020
* Add e2e test workflow

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Add local_setup sourcing

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Add retry logic to tests

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Add requirements.txt

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Add shell script for ease of use

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Update e2e test workflow to run shell script

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Add external em dependency

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Update e2e_test.yml

- remove python action
- add debug ssh action

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Remove ssh debug

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Simplify test

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Fix linting

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Remove em dependency

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Fix return value on success
Simplify process kill

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Fix uncrustify linting (#89)

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Move dependencies to test
Update test script

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Update package.xml

Fix comment

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Refactor to use retrying module

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Update e2e test docstring
Address review comment

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>

* Update script docstring

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants