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

Weekly Coverage Runs to coveralls.io #3055

Merged
merged 4 commits into from
Jul 21, 2021

Conversation

argakiig
Copy link
Contributor

@argakiig argakiig commented Nov 19, 2020

Coverage runs scheduled for 1am UTC Sunday
Allow for Manual Coverage runs as well if necessary
[build|run]_tests custom targets added with run_tests being a wrapper for ci/test.sh
load_test now properly depends on nano_node and nano_rpc

coverage based targets enabled with LCOV=1 environment variable currently only gcc, may extend later for clang and others
generate_[coverage|html] custom targets for generating lcov and html formatted reports
coveralls_upload target requires cpp_coveralls python module in addition to lcov

matrixes TEST_USE_ROCKSDB in a single tests.yml for less duplication

coverage as done via pr only currently shown here https://coveralls.io/github/nanocurrency/nano-node
when merged coverage will update weekly against develop at almost 50 min a run for the longer of the two variants

@argakiig argakiig self-assigned this Nov 19, 2020
@argakiig argakiig added static-analysis tool Introduces or updates a tool or command universe This item indicates the need for or supplies changes caused by external factors labels Nov 19, 2020
@argakiig argakiig changed the title Run Weekly Coverage Runs Weekly Coverage Runs to coveralls.io Nov 19, 2020
@argakiig argakiig marked this pull request as draft November 30, 2020 16:30
@argakiig argakiig force-pushed the weekly_coveralls branch 3 times, most recently from b821feb to d1232ce Compare March 6, 2021 03:58
@argakiig argakiig marked this pull request as ready for review March 12, 2021 02:24
@zhyatt zhyatt added this to the V22.0 milestone Mar 31, 2021
@zhyatt zhyatt removed the request for review from wezrule April 14, 2021 16:50
@argakiig argakiig dismissed a stale review via 8d144aa April 26, 2021 04:21
@argakiig argakiig force-pushed the weekly_coveralls branch 3 times, most recently from 781573d to b404e61 Compare April 26, 2021 05:05
@zhyatt zhyatt modified the milestones: V22.0, V23.0 May 10, 2021
@argakiig argakiig merged commit b687356 into nanocurrency:develop Jul 21, 2021
@argakiig argakiig deleted the weekly_coveralls branch July 21, 2021 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
static-analysis tool Introduces or updates a tool or command universe This item indicates the need for or supplies changes caused by external factors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants