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 ubuntu 16.04 from self-tests, out of support #127

Merged
merged 1 commit into from Aug 8, 2021

Conversation

stronk7
Copy link
Member

@stronk7 stronk7 commented Aug 7, 2021

And being removed from the GHA setup-php

https://github.com/shivammathur/setup-php
shivammathur/setup-php#452

So, jumping to 20.04 and also adding some mising php80 runs.

@stronk7
Copy link
Member Author

stronk7 commented Aug 7, 2021

OT: had to rerun the GHA workflow because one of the jobs failed (crash). It seems that there is some randomness right now.

See https://github.com/stronk7/moodle-plugin-ci/actions?query=branch%3Abye_ubuntu_16_04++ , it happened the same (different branches affected) while preparing the PR (I did some changes disabling coverage to see if it was related or no).

At the end, it passes... but not nice... going to rerun the workflow a number of times.

@stronk7
Copy link
Member Author

stronk7 commented Aug 7, 2021

Yeah, have run it 3 times:

  • 1 passed
  • 1 failed the php80 master job.
  • 1 failed the php80 master and 311 jobs.

I'm going to remove the new php80 jobs from the matrix, so this will be, exclusively, about to move to 20.04

Copy link
Member

@kabalin kabalin left a comment

Choose a reason for hiding this comment

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

Thanks @stronk7, looks good to me.

@kabalin kabalin merged commit e7fca18 into moodlehq:master Aug 8, 2021
@stronk7 stronk7 deleted the bye_ubuntu_16_04 branch August 8, 2021 16:36
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

2 participants