Skip to content
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

puppet6 is released, use it with acceptance tests #501

Merged
merged 2 commits into from
Oct 5, 2018

Conversation

Dan33l
Copy link
Member

@Dan33l Dan33l commented Oct 4, 2018

Now puppet6 is realsed it is better to not use nightly as default.

@bastelfreak
Copy link
Member

Should we instead add another travis slot for this? @dhollinger recommended this. We could test on puppet6 + puppet6-nightlies to catch issues early.

@Dan33l Dan33l force-pushed the puppet6-without-nightly branch from c804dbd to 3241f9d Compare October 4, 2018 09:16
@Dan33l
Copy link
Member Author

Dan33l commented Oct 4, 2018

Yes i agree. I probably misunderstood a resource limit about CI slots.
Done

@ekohl
Copy link
Member

ekohl commented Oct 4, 2018

IMHO the nightly testing could be good if we have cron runs of modules but AFAIK we don't do so now. That'd mean we have to get a PR in and just get lucky to see something break.

@bastelfreak
Copy link
Member

@ekohl I would like to have that, but who has time for that to configure it :(. I think nevertheless we should test the nightlies and afterwards try to enable the crons in travis

@Dan33l Dan33l force-pushed the puppet6-without-nightly branch 2 times, most recently from 516db70 to bbe4479 Compare October 4, 2018 12:02
@Dan33l Dan33l changed the title puppet6 is released, nightly removed puppet6 is released Oct 4, 2018
@Dan33l Dan33l changed the title puppet6 is released puppet6 is released, use it with acceptance tests Oct 4, 2018
@bastelfreak bastelfreak merged commit c13365b into voxpupuli:master Oct 5, 2018
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.

3 participants