build: add GH Actions CI #243

Merged
Bogdanp merged 1 commits from gh-actions into master 3 years ago
Bogdanp commented 3 years ago (Migrated from github.com)

I made this change thinking there was no CI in place already, but I noticed the Travis config just now so feel free to reject this. Then again, it might make sense to keep both given Travis' upcoming billing changes.

I made this change thinking there was no CI in place already, but I noticed the Travis config just now so feel free to reject this. Then again, it might make sense to keep both given Travis' [upcoming billing changes](https://blog.travis-ci.com/2020-11-02-travis-ci-new-billing).
mbutterick commented 3 years ago (Migrated from github.com)

Thanks! I had been thinking I needed to migrate away from Travis, but I hadn’t looked into the details.

Thanks! I [had been thinking](https://github.com/mbutterick/pollen-users/issues/75) I needed to migrate away from Travis, but I hadn’t looked into the details.
mbutterick commented 3 years ago (Migrated from github.com)

Can your GH Actions script test Racket 6.3 and later?
Is there any reason for me to keep Travis?

Can your GH Actions script test Racket 6.3 and later? Is there any reason for me to keep Travis?
Bogdanp commented 3 years ago (Migrated from github.com)

I think it can currently install Rackets as far back as 6.5, but I should be able to modify it to support 6.4 and 6.3. The installer URL structure is different prior to 6.5 so that's why those versions are not currently supported.

I only went back to 7.4 in my PR because that's as far back as the CS releases go. If you like, I can make a new PR to get it to test on 6.5 and up and have it conditionally test the CS variant for the versions where it's available. If you want to keep supporting 6.3 and 6.4 in CI, I can try to add support for those to setup-racket next weekend.

I don't think there's any reason in particular to keep Travis around.

I think it can currently install Rackets as far back as 6.5, but I should be able to modify it to support 6.4 and 6.3. The installer URL structure is different prior to 6.5 so that's why those versions are not currently supported. I only went back to 7.4 in my PR because that's as far back as the CS releases go. If you like, I can make a new PR to get it to test on 6.5 and up and have it conditionally test the CS variant for the versions where it's available. If you want to keep supporting 6.3 and 6.4 in CI, I can try to add support for those to `setup-racket` next weekend. I don't think there's any reason in particular to keep Travis around.
mbutterick commented 3 years ago (Migrated from github.com)

If 6.5 is the easier target, then I can live with that. I can drop support for 6.3 and 6.4. The price of progress 🤘

If 6.5 is the easier target, then I can live with that. I can drop support for 6.3 and 6.4. The price of progress 🤘
mbutterick commented 3 years ago (Migrated from github.com)

Thanks for setup-racket!

Thanks for `setup-racket`!
The pull request has been merged as 62b19a07d8.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b gh-actions master
git pull origin gh-actions

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff gh-actions
git push origin master
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: mbutterick/pollen#243
Loading…
There is no content yet.