Skip to content

add Stickler CI config#564

Merged
wholmgren merged 3 commits into
pvlib:masterfrom
wholmgren:master
Sep 6, 2018
Merged

add Stickler CI config#564
wholmgren merged 3 commits into
pvlib:masterfrom
wholmgren:master

Conversation

@wholmgren

Copy link
Copy Markdown
Member

I'd like to try using Stickler CI service with pvlib. https://stickler-ci.com

Stickler CI will automatically review pull requests and comment on style issues. I configured it on my fork and made this example pull request: wholmgren#5

Pros:

  • Maintainers don't have to spend as much time on style issues
  • It won't accidentally overlook issues
  • People can get frustrated at a bot that nags them instead of at me

Cons:

  • It's an extra notification clogging your inbox
  • Maybe it makes pull requests a little harder to read

@wholmgren wholmgren added this to the 0.6.0 milestone Sep 6, 2018
@wholmgren wholmgren merged commit fec0679 into pvlib:master Sep 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant