Skip to content
This repository has been archived by the owner on Feb 24, 2024. It is now read-only.

buffalo/cmd: make len comparison more clear #1212

Merged
merged 3 commits into from Aug 6, 2018
Merged

buffalo/cmd: make len comparison more clear #1212

merged 3 commits into from Aug 6, 2018

Conversation

quasilyte
Copy link
Contributor

len call never returns negative values, so checking against 0 is enough.

Found using https://go-critic.github.io/overview#sloppyLen-ref

stanislas-m and others added 2 commits August 3, 2018 19:14
Added Zhorty because of my Generous Backer status on https://www.patreon.com/buffalo.
len call never returns negative values, so checking against 0 is enough.

Found using https://go-critic.github.io/overview#sloppyLen-ref
@markbates markbates changed the base branch from master to development August 4, 2018 23:54
@markbates markbates added this to the v0.13.0 milestone Aug 6, 2018
@markbates markbates merged commit 52242d6 into gobuffalo:development Aug 6, 2018
@quasilyte quasilyte deleted the quasilyte/sloppyLen branch August 6, 2018 13:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants