Skip to content

Run CI regularly twice a month#576

Merged
peterzahemszky merged 4 commits into
masterfrom
574-regular-ci
Nov 23, 2021
Merged

Run CI regularly twice a month#576
peterzahemszky merged 4 commits into
masterfrom
574-regular-ci

Conversation

@peterzahemszky

@peterzahemszky peterzahemszky commented Nov 19, 2021

Copy link
Copy Markdown
Contributor

Run CI regularly twice a month to discover dependency issues (like #574) earlier. Currently they're arbitrarily scheduled for the 7th and 22nd days of the month.

From the documentation of on.schedule:

Scheduled workflows run on the latest commit on the default or base branch.

Closes #549

export LC_ALL=en_US.UTF-8
export LANG=en_US.UTF-8
make pythontest No newline at end of file
make pythontest

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This extra newline character at the end of the file is unrelated to the rest of the PR

@flongford

Copy link
Copy Markdown
Contributor

FYI - merging this PR will also close #549

@flongford flongford left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggestion: change name on each workflow to include OS

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #576 (0d95f81) into master (43943f4) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #576   +/-   ##
=======================================
  Coverage   95.35%   95.35%           
=======================================
  Files          92       92           
  Lines        4305     4305           
  Branches      277      277           
=======================================
  Hits         4105     4105           
  Misses        140      140           
  Partials       60       60           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 43943f4...0d95f81. Read the comment docs.

@flongford flongford left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@peterzahemszky peterzahemszky merged commit 6231b87 into master Nov 23, 2021
@peterzahemszky peterzahemszky deleted the 574-regular-ci branch November 23, 2021 10:40
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.

Consider Cron jobs in CI to help maintenance

3 participants