gh: daily validation of the ELL compatibility #308
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
We recently had some issues because the ELL API has been broken without notice. Simply adding a cron to run the tests once a day during the week should tell us when an incompatible change has been introduced on ELL side, so we can get a bit of time to prepare a fix and a new release. Link: multipath-tcp#302 Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
The previous version is deprecated, each job gets this warning: The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/ No further changes needed: we use the public runners with ubuntu-latest. Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
The previous version is deprecated, each job using it gets this warning: The following actions uses a deprecated Node.js version and will be forced to run on node20: coverallsapp/github-action@master. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/ We just have to use 'file' instead of 'path-to-lcov' which is deprecated. No further changes needed: we use the public runners with ubuntu-latest. Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
Pull Request Test Coverage Report for Build 11502994362Details
💛 - Coveralls |
|
I don't think any reviews are needed here: just some simple modifications of the CI config, the behaviour is not supposed to be modified, only one new trigger. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We recently had some issues because the ELL API has been broken without notice.
Simply adding a cron to run the tests once a day during the week should tell us when an incompatible change has been introduced on ELL side, so we can get a bit of time to prepare a fix and a new release.
While at it, also switch to newer versions of some deprecated GH actions.