Skip to content

Release Candidate 1 for 1.7.2 - #138

Merged
nicklucius merged 28 commits into
masterfrom
dev
Apr 12, 2017
Merged

Release Candidate 1 for 1.7.2#138
nicklucius merged 28 commits into
masterfrom
dev

Conversation

@nicklucius

Copy link
Copy Markdown
Contributor

Version 1.7.2 has several changes:

Performance improvements:

  • By default, 1000 rows at a time were being retreived for each API call while paging through data. Increased to retreive 50,000 rows for each API call. (Increase Paging Capacity #129)

Bug fixes:

Deprecation:

RSocrata's core development team has stated a formal policy to only support the most recent release of R. Until now, RSocrata was tested against the penultimate release of R; however, testing will be limited to the current version of R and the current development release. The project's contributing guidelines have been updated to reflect that accepted changes to RSocrata must pass tests on the current and penulimate versions of R.

While RSocrata is only tested on the current and penultimate version, the core development team expects it will work on older versions most of the time. See #132 for more information.

tomschenkjr and others added 28 commits November 16, 2016 22:17
* Added testing for issue #121 -- will fail

* Changed to camel case to accomodate future variable renaming

* Added handling of ISO 8601 timestamps without fractional seconds. See issue #121

* Increased minimum version of R because of `anyNA` function.

* Unformatted dates kept as chr; added warning; added tests

* Added tests for #124 -- will fail

* Handles downloadURL from ls.socrata. Fixes #124
Merge branch 'dev' of https://github.com/Chicago/RSocrata into dev

# Conflicts:
#	DESCRIPTION
Merge branch 'issue129' into dev

# Conflicts:
#	DESCRIPTION
#	tests/testthat/test-all.R
@tomschenkjr tomschenkjr added this to the 1.7.2 milestone Apr 11, 2017
@coveralls

coveralls commented Apr 11, 2017

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.07%) to 96.341% when pulling e28d51c on dev into dfa7001 on master.

@nicklucius
nicklucius merged commit eae61b5 into master Apr 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants