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.

Sign up for GitHub

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

Jump to bottom

Update CI workflows #243

Merged
merged 5 commits into from
Oct 30, 2023
Merged

Update CI workflows #243

merged 5 commits into from
Oct 30, 2023

Conversation

nickvergessen
Copy link
Member

  • Update supported PHP versions
  • Update supported Nextcloud versions
  • Ensure cancel-parallel is set
  • Don't urn on pushes anymore

@nickvergessen
ci: Turn off builds for pushes and merges to master/main/stable*
2ab53c4 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
ci: Cancel parallel cypress excutions
ce9a0ca 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix: Run default CI on PHP 8.2 because PHP 8.0 is EOL
fe575d0 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix: Run default lint against 8.3 as well as server 28 supports it
f4b032f 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix: Remove stable25 from actions and add stable28 already
353c785 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the ci/noid/improve-handling branch from 2d73af2 to 353c785 Compare October 30, 2023 11:11
@susnux
Copy link
Contributor

susnux commented Oct 30, 2023

Should be fine as project code coverage (calculated on push to branch) is not part of our templates, so apps have to adjust phpunit tests either way.

@ChristophWurst ChristophWurst removed their request for review October 30, 2023 11:16
@nickvergessen nickvergessen merged commit 3e4181b into master Oct 30, 2023
3 checks passed
@nickvergessen nickvergessen deleted the ci/noid/improve-handling branch October 30, 2023 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@provokateurin provokateurin provokateurin approved these changes

@miaulalala miaulalala miaulalala approved these changes

@susnux susnux susnux approved these changes

@skjnldsv skjnldsv Awaiting requested review from skjnldsv

@come-nc come-nc Awaiting requested review from come-nc

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nickvergessen @susnux @miaulalala @provokateurin