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

Remove CI testing on Travis #240

Merged
merged 1 commit into from
Sep 15, 2023
Merged

Conversation

stronk7
Copy link
Member

@stronk7 stronk7 commented Sep 15, 2023

To be done after #239

While we still keep moodle-plugin-ci officially supported with Travis, it just so slow (for our current paid plan) that make any branch or PR to take ages to be tested (~1 hour).

So here we are reducing the number of branches being tested to:

  • master (highest PHP version supported).
  • latest stable (highest PHP version supported).
  • oldest stable supported (lowest PHP version supported).

That will save some precious minutes (7-8 per run) making things a little bit more agile, specially given the falling relevance of Travis since a couple of years ago (compared with the raise of GHA).

While we still keep moodle-plugin-ci officially supported
with Travis, it just so slow (for our current paid plan)
that make any branch or PR to take ages to be tested (~1 hour).

So here we are reducing the number of branches being tested to:
- master (highest PHP version supported).
- latest stable (highest PHP version supported).
- oldest stable supported (lowest PHP version supported).

That will save some precious minutes (7-8 per run) making things
a little bit more agile, specially given the falling relevance
of Travis since a couple of years ago (compared with the raise
of GHA).
@codecov
Copy link

codecov bot commented Sep 15, 2023

Codecov Report

Merging #240 (ba34be3) into master (ed8d11b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #240   +/-   ##
=========================================
  Coverage     84.58%   84.58%           
  Complexity      689      689           
=========================================
  Files            74       74           
  Lines          2134     2134           
=========================================
  Hits           1805     1805           
  Misses          329      329           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@stronk7 stronk7 merged commit 3648f5c into moodlehq:master Sep 15, 2023
13 checks passed
@stronk7 stronk7 deleted the reduce_travis_testing branch September 15, 2023 15:37
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