Skip to content
This repository was archived by the owner on Mar 20, 2023. It is now read-only.

CI Updates: live debugging for PRs only + docs deployment#680

Merged
alexsavulescu merged 2 commits intomasterfrom
ciupdates
Oct 27, 2021
Merged

CI Updates: live debugging for PRs only + docs deployment#680
alexsavulescu merged 2 commits intomasterfrom
ciupdates

Conversation

@alexsavulescu
Copy link
Copy Markdown
Contributor

CI_BRANCHES:NEURON_BRANCH=master,

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #680 (a4f1d25) into master (29ef490) will decrease coverage by 0.05%.
The diff coverage is n/a.

❗ Current head a4f1d25 differs from pull request most recent head daad3bc. Consider uploading reports for the commit daad3bc to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #680      +/-   ##
==========================================
- Coverage   55.26%   55.20%   -0.06%     
==========================================
  Files         107      107              
  Lines        8716     8716              
==========================================
- Hits         4817     4812       -5     
- Misses       3899     3904       +5     
Impacted Files Coverage Δ
coreneuron/utils/progressbar/progressbar.cpp 82.95% <0.00%> (-5.69%) ⬇️

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 29ef490...daad3bc. Read the comment docs.

Comment thread .github/workflows/coreneuron-ci.yml Outdated
Co-authored-by: Ioannis Magkanaris <iomagkanaris@gmail.com>
Copy link
Copy Markdown
Contributor

@iomaganaris iomaganaris left a comment

Choose a reason for hiding this comment

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

LGTM

@alexsavulescu alexsavulescu merged commit d6b866f into master Oct 27, 2021
@alexsavulescu alexsavulescu deleted the ciupdates branch October 27, 2021 11:22
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.

4 participants