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

[ci] Run the CI daily on main branch #91

Merged
merged 1 commit into from
Nov 27, 2023
Merged

[ci] Run the CI daily on main branch #91

merged 1 commit into from
Nov 27, 2023

Conversation

SamuelRiedel
Copy link
Member

@SamuelRiedel SamuelRiedel commented Nov 13, 2023

This has two benefits: First, we ensure that the main branch works even if the runners/dependencies update. Second, GitHub's caches are scoped. They only work among commits of the same branch, except from the default branch. (https://github.com/actions/cache#cache-scopes) Therefore, if we ensure that the main branch is always cached, we should always hit the caches and have quick turnaround times on all our branches and PRs.

Checklist

  • Automated tests pass
  • Changelog updated
  • Code style guideline is observed

This has two benefits: First, we ensure that the main branch works even
if the runners/dependencies update. Second, GitHub's caches are scoped.
They only work among commits of the same branch, except from the default
branch. (https://github.com/actions/cache#cache-scopes) Therefore, if we
ensure that the main branch is always cached, we should always hit the
caches and have quick turnaround times on all our branches and PRs.
Copy link
Collaborator

@sermazz sermazz left a comment

Choose a reason for hiding this comment

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

Cool! :D Looks good to me

@SamuelRiedel SamuelRiedel merged commit a341a7d into main Nov 27, 2023
62 checks passed
@SamuelRiedel SamuelRiedel deleted the sriedel/ci branch November 27, 2023 08:27
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