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

Update GitHub Actions CI #24

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

biow0lf
Copy link
Contributor

@biow0lf biow0lf commented Mar 9, 2023

List of changes:

  1. Add Ruby 3.2 and head.
  2. Use bundler-cache for install and cache gems
  3. Run tests by cron at "0 21 * * 6". Each week.
  4. Add Dependabot for updating GitHub Actions
  5. Update links from master to main
  6. Update travis badge. png->svg.

Note: this repo has master and main branches. Looks like, travis uses master branch and should be updated. Also, master has commits that missed in main. So, let's merge master in main and remove it (master).

branches:
- main
schedule:
- cron: "0 21 * * 6"

Choose a reason for hiding this comment

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

I don't think we want to run this on a cron and burn up CI minutes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@geoffharcourt for open source projects GitHub is free. And we can and we should do it.

And in this case 0 21 * * 6 mean at 21:00 on each Saturday.. Sounds good. One per week run CI to check that everything still works.

Copy link

@geoffharcourt geoffharcourt left a comment

Choose a reason for hiding this comment

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

LGTM. I think we should address whether or not we want a cron running unattended given how little activity there is on the repo in genearl.

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.

None yet

3 participants