Skip to content

Conversation

@MichaelChirico
Copy link
Contributor

Identified by lintr::seq_linter().

I didn't see enough input validation to ensure that a length-0 input doesn't wind up running 1:0 accidentally in these cases, hence seq_along()/seq_len() should be preferred.

@edzer edzer merged commit effea0a into r-spatial:main Jun 22, 2023
@MichaelChirico MichaelChirico deleted the avoid-1-to branch June 22, 2023 18:02
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.

2 participants