Skip to content

Conversation

@lorenzwalthert
Copy link
Collaborator

result should at least be parsable. Closes #892.

result should at least be parsable
@github-actions
Copy link
Contributor

github-actions bot commented Jan 2, 2022

This is how benchmark results would change (along with a 95% confidence interval in relative change) if 9a3558b is merged into main:

  •   :ballot_box_with_check:cache_applying: 27.2ms -> 27.2ms [-1.6%, +1.38%]
  •   :ballot_box_with_check:cache_recording: 954ms -> 958ms [-0.84%, +1.63%]
  •   :ballot_box_with_check:without_cache: 2.59s -> 2.59s [-1.47%, +1.72%]

Further explanation regarding interpretation and methodology can be found in the documentation.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 3, 2022

This is how benchmark results would change (along with a 95% confidence interval in relative change) if c74978b is merged into main:

  •   :ballot_box_with_check:cache_applying: 29.3ms -> 28.7ms [-3.88%, +0.07%]
  •   :ballot_box_with_check:cache_recording: 1.01s -> 1.02s [-1.47%, +3.51%]
  •   :ballot_box_with_check:without_cache: 2.74s -> 2.69s [-4.04%, +0.35%]

Further explanation regarding interpretation and methodology can be found in the documentation.

@codecov-commenter
Copy link

Codecov Report

Merging #895 (9369002) into main (1744cb3) will increase coverage by 0.67%.
The diff coverage is 96.61%.

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

@@            Coverage Diff             @@
##             main     #895      +/-   ##
==========================================
+ Coverage   89.65%   90.33%   +0.67%     
==========================================
  Files          47       47              
  Lines        2514     2546      +32     
==========================================
+ Hits         2254     2300      +46     
+ Misses        260      246      -14     
Impacted Files Coverage Δ
R/relevel.R 46.37% <60.00%> (+0.22%) ⬆️
R/communicate.R 92.30% <100.00%> (ø)
R/detect-alignment-utils.R 90.90% <100.00%> (+0.28%) ⬆️
R/roxygen-examples.R 98.43% <100.00%> (+0.05%) ⬆️
R/rules-tokens.R 100.00% <100.00%> (ø)
R/token-create.R 96.07% <100.00%> (+0.16%) ⬆️
R/transform-files.R 97.24% <100.00%> (+0.22%) ⬆️
R/zzz.R 39.53% <100.00%> (+39.53%) ⬆️

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 f672907...71aa4c6. Read the comment docs.

@lorenzwalthert lorenzwalthert merged commit bfa8309 into r-lib:main Jan 3, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Jan 3, 2022

This is how benchmark results would change (along with a 95% confidence interval in relative change) if 97f549a is merged into main:

  •   :ballot_box_with_check:cache_applying: 27.5ms -> 27.7ms [-0.29%, +1.78%]
  •   :ballot_box_with_check:cache_recording: 958ms -> 966ms [-0.41%, +1.93%]
  •   :ballot_box_with_check:without_cache: 2.55s -> 2.54s [-1.17%, +0.45%]

Further explanation regarding interpretation and methodology can be found in the documentation.

@lorenzwalthert lorenzwalthert deleted the issue-892 branch January 3, 2022 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Better roundtrip validation

2 participants