Skip to content

Conversation

@topepo
Copy link
Member

@topepo topepo commented Feb 25, 2019

Make sure all comments in doc files and stop

  • have functions as foo() instead of foo

  • use backticks for argument names (e.g. not 'bar')

@codecov-io
Copy link

codecov-io commented Feb 25, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@7e69c55). Click here to learn what that means.
The diff coverage is 18.18%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #151   +/-   ##
=========================================
  Coverage          ?   73.17%           
=========================================
  Files             ?       35           
  Lines             ?     2587           
  Branches          ?        0           
=========================================
  Hits              ?     1893           
  Misses            ?      694           
  Partials          ?        0
Impacted Files Coverage Δ
R/arguments.R 66.19% <ø> (ø)
R/engines.R 80% <ø> (ø)
R/mars.R 75.72% <ø> (ø)
R/mlp.R 77.33% <ø> (ø)
R/boost_tree.R 84.46% <ø> (ø)
R/translate.R 73.68% <ø> (ø)
R/rand_forest.R 77.61% <ø> (ø)
R/svm_poly.R 83.07% <ø> (ø)
R/surv_reg.R 50.7% <ø> (ø)
R/linear_reg.R 73.86% <ø> (ø)
... and 15 more

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 7e69c55...4f2e670. Read the comment docs.

@topepo topepo merged commit 5bc61d8 into master Feb 25, 2019
@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2021

This pull request has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 8, 2021
@hfrick hfrick deleted the doc-consistency-changes branch September 17, 2021 14:27
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.

3 participants