Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- Andrew Lyons
- Unrelated, can we please discuss MDL-74833 and MDL-74835?
- I think that we should just drop support for Travis. We do not test with it any more, so it's just going to be a pain to maintain.
- Kevin Bot
- MDL-74833 - Update travis integration from bionic (18.04) to focal (20.04)
- Status: Waiting for peer review
- Reporter: Daniel Ziegenberg
- Assignee: Daniel Ziegenberg
- MDL-74835 - Update nodejs version in travis integration to lts/gallium
- Status: Waiting for integration review
- Reporter: Daniel Ziegenberg
- Assignee: Daniel Ziegenberg
- ...
- ...
- Jun Pataleta
- I think that we should just drop support for Travis. We do not test with it any more, so it's just going to be a pain to maintain.
- If we're dropping travis support, are we gonna drop it for all security and stable branches?
- If so, then it doesn't make sense to spend time updating .travis.yml
- But if dropping Travis support will only be for master then maybe we can entertain the patch for 74835
- I'm okay too with dropping Travis. I think Eloy still uses it though but not sure
- Kevin Bot
- ...
- ...
- Eloy Lafuente (stronk7)
- About Travis we (HQ, also me, personal) have paid accounts. Not all products have moved away from it yet.
- And, more precisely for core, I only use the integration to:
- Verify it continues working (when I submit a patch).
- Run weekly windows runs (39_STABLE and up). But already have moved those to GHA.
- So, with my/our only use being 1) we certainly don't need it in core (again, we need it in some suproducts).
- What I'm not sure is if people out there will be using it or no, because certainly it just works.
- Andrew Lyons
- Well does it just work..?
- Eloy Lafuente (stronk7)
- I think so. One time every few months we have to update it with this or that detail, but I'd not say it's a nightmare to maintain.
- I mean, the runs I get from my branches work ok:
- https://app.travis-ci.com/github/stronk7/moodle/branches
- The point, as commented, is if there are people out there using it with complex plugin aggregations or things like that. But, if that happens, surely they will be using custom .travis file already, heh.
- So I don't oppose to remove it from 4.1 and up.
Advertisement
Add Comment
Please, Sign In to add comment