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

Add workflow for GitHub Actions #867

Merged
merged 5 commits into from
Sep 20, 2022
Merged

Add workflow for GitHub Actions #867

merged 5 commits into from
Sep 20, 2022

Conversation

btcbee
Copy link
Contributor

@btcbee btcbee commented Aug 12, 2022

This is a basic GHA workflow to get the automatic test suite running again. I'm sure there's room for optimization, but this should be a good start.

Sample test run: https://github.com/btcbee/tmuxinator/actions/runs/2849166648

Should the .travis.yml file be removed in the same PR?
@ethagnawl

Fixes #850

bash autogen.sh
bash ./configure && make && sudo make install
cd ..
tmux -V
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No automatic check here to verify that the version built matches the version requested, because sometimes the prerelease was not removed before the release was cut (e.g. v3.1). 😭

@ethagnawl
Copy link
Member

This is great, @btcbee. Thanks for putting it together! I'll try to give this a proper review and get it running later this week.

Also, yes, I'd say the Travis config file should be removed as part of this integration.

@btcbee btcbee force-pushed the master branch 3 times, most recently from 4975340 to 8ed3afe Compare August 17, 2022 04:16
@btcbee
Copy link
Contributor Author

btcbee commented Aug 17, 2022

  • Removed .travis.yml
  • Updated build status badge in REAME.md to reference GHA badge. This currently appears broken because the correct link doesn't match the forked repo. Before the final change to set the correct repo name, the badge looked like this:

passing

Fix typo
@ethagnawl
Copy link
Member

This looks great, @btcbee. Thanks again.

Does anyone else have any thoughts, @tmuxinator/tmuxinator?

@ethagnawl ethagnawl merged commit 7aa8092 into tmuxinator:master Sep 20, 2022
@ethagnawl
Copy link
Member

Thanks again, @btcbee!

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.

Migrate away from Travis for CI
2 participants