Skip to content
This repository has been archived by the owner on Sep 9, 2022. It is now read-only.

add chris as contributor #13

Merged
merged 1 commit into from Jan 16, 2019
Merged

Conversation

crew102
Copy link
Collaborator

@crew102 crew102 commented Jan 16, 2019

As per discussion in #12

@codecov-io
Copy link

codecov-io commented Jan 16, 2019

Codecov Report

Merging #13 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #13   +/-   ##
=======================================
  Coverage   77.88%   77.88%           
=======================================
  Files           8        8           
  Lines         104      104           
=======================================
  Hits           81       81           
  Misses         23       23

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 9354da8...ff9d1ef. Read the comment docs.

@sckott
Copy link
Contributor

sckott commented Jan 16, 2019

LGTM

gave you push access to repo. Early on I'd ask that you still do branching and PR's, but can push directly for smaller things.

@sckott sckott added this to the v0.5 milestone Jan 16, 2019
@sckott sckott merged commit 0d1869f into ropensci-archive:master Jan 16, 2019
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