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

Add license and other requirements for CRAN #20

Merged
merged 2 commits into from Feb 5, 2019

Conversation

Projects
None yet
2 participants
@greebie
Copy link
Collaborator

greebie commented Feb 5, 2019

Some minor changes to set up for cran (e.g. add documentation for default).
image

@greebie

This comment has been minimized.

Copy link
Collaborator

greebie commented Feb 5, 2019

This version has been submitted to CRAN as archivr-0.0.1. Once this is merged, I will push a release to complete the project.

LICENSE Outdated
@@ -0,0 +1,19 @@
#' Copyright <2019> <Qualitative Data Repository, Syracuse University>

This comment has been minimized.

@adam3smith

adam3smith Feb 5, 2019

Contributor

Any reason you're using the #' roxygen comments? Since this is in a separate file, the license can just be plain text, no?

This comment has been minimized.

@greebie

greebie Feb 5, 2019

Collaborator

No reason to use comments I don't think.

This comment has been minimized.

@adam3smith

adam3smith Feb 5, 2019

Contributor

Will it mess with your CRAN submission if I take these out? Otherwise this is good to go.

This comment has been minimized.

@greebie

greebie Feb 5, 2019

Collaborator

No I don't think so. Once it's accepted (or not) I can repair it accordingly.

@adam3smith adam3smith merged commit 0dcf54c into master Feb 5, 2019

@greebie greebie referenced this pull request Feb 5, 2019

Closed

Launch v0.0.1 #17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment