Skip to content

Comments

R/latlong2fips.R: sprintf w/o warning, check json$Block$FIPS#145

Open
WillForan wants to merge 1 commit intoRPVote:masterfrom
WillForan:master
Open

R/latlong2fips.R: sprintf w/o warning, check json$Block$FIPS#145
WillForan wants to merge 1 commit intoRPVote:masterfrom
WillForan:master

Conversation

@WillForan
Copy link

Thanks for opening a PR!

  • Have you followed the guidelines in our Contributing Guide?
  • Ensured you are not committing large data files or other sensitive information (e.g. voter files)

In 2-3 sentences or a brief list, describe changes implemented in this PR

use sprintf with numeric args to avoid warning, test length of returned json instead of string. keep roiw_id and instead set FIP to 'CONVERGE-FAIL"

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant