-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[feature] generate .travis.yml #191
Conversation
Codecov Report
@@ Coverage Diff @@
## master #191 +/- ##
==========================================
+ Coverage 69.26% 69.35% +0.09%
==========================================
Files 8 10 +2
Lines 693 744 +51
==========================================
+ Hits 480 516 +36
- Misses 129 137 +8
- Partials 84 91 +7
Continue to review full report at Codecov.
|
|
||
func (p *Plan) buildTravisCI(c *config.Config) TravisCI { | ||
if p.Accounts == nil { | ||
panic("buildTravisCI must be run after buildAccounts") |
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.
why not return an error instead?
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.
Only thing I wonder is if we should allow users to have their own travis logic as well. Might also be worth adding a bit to the readme or something about this feature.
@edulop91 good idea |
Auto-generate a .travis-ci.yml based on fogg.json.
This introduced a new top-level config in fogg.json for configuring things about travis ci.
When enabled it will use the accounts config to set up profiles and then bucket all the components and modules into a configured number of buckets.