Skip to content
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

v3.1.2 #252

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 22 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
5 changes: 4 additions & 1 deletion .Rbuildignore
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,8 @@
^docs$
^CODE_OF_CONDUCT\.md$
^_pkgdown\.yaml$
^CHANGELOG\.md$
^NEWS\.md$
^cran-comments\.md$
^doc$
^Meta$
^CRAN-SUBMISSION$
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -9,3 +9,6 @@ inst/web
.*.swp
.*.swo
.DS_Store
civis.Rcheck/
/doc/
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm confused that /doc/ is listed here but this PR adds a bunch of files in that directory. Should this be doc/ and meta/?

At any rate, I think Meta/vignette.rds and doc/* should be removed from this branch and squashed away when merging.

/Meta/
3 changes: 3 additions & 0 deletions CRAN-SUBMISSION
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
Version: 3.1.0
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this should be removed and put in the .gitignore. See https://stackoverflow.com/a/72631281. It doesn't seem unreasonable to leave it in, but this particular file is out of date.

Date: 2023-02-14 21:28:04 UTC
SHA: b389b98a9c1914f99f9d68509f62a43fd907b5d5
12 changes: 7 additions & 5 deletions DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
Package: civis
Title: R Client for the 'Civis Platform API'
Version: 3.0.0
Version: 3.1.2
Authors@R: c(
person("Patrick", "Miller", email = "[email protected]", role = c("cre", "aut")),
person("Peter", "Cooman", email = "[email protected]", role = c("cre", "ctb")),
person("Patrick", "Miller", email = "[email protected]", role = "aut"),
person("Keith", "Ingersoll", email = "[email protected]", role = "aut"),
person("Bill", "Lattner", email = "[email protected]", role = "ctb"),
person("Anh", "Le", email = "[email protected]", role = "ctb"),
Expand All @@ -13,14 +14,14 @@ Authors@R: c(
person("Elizabeth", "Sander", email = "[email protected]", role = "ctb"),
person("Madison", "Hobbs", email = "[email protected]", role = "ctb"),
person("Anna", "Bladey", email = "[email protected]", role = "ctb"),
person("Sahil", "Shah", email = "[email protected]", role = "ctb"))
person("Sahil", "Shah", email = "[email protected]", role = "ctb"),
person("Bryan", "Baird", email = "[email protected]", role = "ctb"))
Description: A convenient interface for making
requests directly to the 'Civis Platform API' <https://www.civisanalytics.com/platform/>.
Full documentation available 'here' <https://civisanalytics.github.io/civis-r/>.
Depends:
R (>= 3.2.0)
License: BSD_3_clause + file LICENSE
LazyData: true
URL: https://github.com/civisanalytics/civis-r
BugReports: https://github.com/civisanalytics/civis-r/issues
Imports:
Expand All @@ -35,14 +36,15 @@ Suggests:
feather,
ggplot2,
knitr,
lifecycle,
rmarkdown,
roxygen2,
mockery,
R.utils,
rstudioapi,
testthat,
yaml
RoxygenNote: 7.1.0
RoxygenNote: 7.2.2
VignetteBuilder: knitr
Collate:
'await.R'
Expand Down
Binary file added Meta/vignette.rds
Binary file not shown.
Loading