Skip to content

(maint) Use GitHub actions instead of Travis and Appveyor CI#288

Merged
jpogran merged 3 commits intopuppetlabs:mainfrom
glennsarti:add-gh-actions
Jan 5, 2021
Merged

(maint) Use GitHub actions instead of Travis and Appveyor CI#288
jpogran merged 3 commits intopuppetlabs:mainfrom
glennsarti:add-gh-actions

Conversation

@glennsarti
Copy link
Contributor

@glennsarti glennsarti commented Jan 5, 2021

Closes #287

Due to Travis CI changing its CI limits it is no longer functioning for this
project. This commit:

  • Migrates both Travis and Appveyor CI to GitHubActions
  • Uses a small PowerShell script to generate the test matrix due to the
    limitations of Workflow YAML
  • Adds a build and artifact test phase, with a limited retention period

jpogran and others added 3 commits January 5, 2021 15:13
Due to Travis CI changing its CI limits it is no longer functioning for this
project.  This commit:

* Migrates both Travis and Appveyor CI to GitHubActions
* Uses a small PowerShell script to generate the test matrix due to the
  limitations of Workflow YAML
* Adds a build and artifact test phase, with a limited retention period
Rubocop and line ending checks are just terrible.  Instead just ignore them for
rubocop rules.
Previously the acceptance tests were intermittently failing on Windows GitHub
Actions runners. I really don't like this. But unfortunately the workspace
symbol loading is asynchronous so you don't _really_ know what it's available.
We could add another Request or extend puppet/getVersion to emit whether the
workspace has been loaded yet.  In the meantime just sleep a long time and hope
it's loaded by then.
@glennsarti glennsarti requested a review from jpogran as a code owner January 5, 2021 07:22
@glennsarti glennsarti requested review from jpogran and removed request for jpogran January 5, 2021 07:22
@glennsarti glennsarti self-assigned this Jan 5, 2021
@glennsarti
Copy link
Contributor Author

Note that the Travis and AppVeyor will fail as it no longer has a configuration file. This is expected.

@glennsarti
Copy link
Contributor Author

This is a rewrite of #287 by @jpogran

@glennsarti
Copy link
Contributor Author

I don't have rights to remove the Travis and Appveyor integrations.

@jpogran jpogran merged commit fa980bd into puppetlabs:main Jan 5, 2021
@glennsarti glennsarti deleted the add-gh-actions branch January 19, 2021 06:54
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.

2 participants