-
Notifications
You must be signed in to change notification settings - Fork 144
Conversation
Looks awesome 🚀 |
3d44d4b
to
52868cb
Compare
- When using Travis-CI and a public GitHub repository the Codecov token isn’t needed. Yay 🎉
- Makes it possible to enable or disable certain steps if needed.
52868cb
to
54c25ec
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great 👍 , just some minor doc issues.
branches: | ||
only: | ||
- master | ||
- /^release-.*$/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
desc 'Run all CI jobs' | ||
desc 'Lint the source code and other linteable artifacts' | ||
lane :lint do | ||
# pod_lib_lint |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this is coming later?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep! Depended on your podspec being merged. Will uncomment it in a subsequent PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome 👍
desc '' | ||
desc '- LINT: Whether the sources and certain artifacts should be linted,' | ||
desc ' defaults to `YES`.' | ||
desc '- RUN_TESTS: Whether unit tests should be ran, defaults to `YES`.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"should be run", "ran" is in the past :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’ll fix that in my subsequent PR (will push very soon).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright, no problems!
|
||
defaults to `YES`. | ||
|
||
- RUN_TESTS: Whether unit tests should be ran, defaults to `YES`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same comment here as for the in-code docs. "Run" instead of "ran".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixing in the next one!
Change to use Travis-CI.
I think this might work, a bit hard to test until we go public and integrate Travis-CI.