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

Fail on extension / tool installation failure #4365

Merged
merged 1 commit into from
Oct 21, 2020

Conversation

greg0ire
Copy link
Member

Q A
Type improvement
BC Break no
Fixed issues n/a

Summary

This leverages a new feature of shivammathur/setup-php that allows to
fail the build if an extension or tool fails to install.

@morozov
Copy link
Member

morozov commented Oct 20, 2020

Looks good.

This leverages a new feature of shivammathur/setup-php that allows to
fail the build if an extension or tool fails to install.
@greg0ire greg0ire marked this pull request as ready for review October 20, 2020 14:34
morozov
morozov previously approved these changes Oct 20, 2020
@greg0ire
Copy link
Member Author

Should I retarget to 2.12.x or will there be other versions on the 2.11.x branch?

@morozov
Copy link
Member

morozov commented Oct 20, 2020

Yeah. I don't anticipate releasing 2.11.x anymore unless some other regressions are reported.

@greg0ire greg0ire changed the base branch from 2.11.x to 2.12.x October 20, 2020 18:26
@greg0ire greg0ire dismissed morozov’s stale review October 20, 2020 18:26

The base branch was changed.

@morozov morozov added this to the 2.12.0 milestone Oct 20, 2020
@greg0ire greg0ire closed this Oct 20, 2020
@greg0ire greg0ire reopened this Oct 20, 2020
@greg0ire greg0ire closed this Oct 21, 2020
@greg0ire greg0ire reopened this Oct 21, 2020
@greg0ire greg0ire merged commit 9350d2a into doctrine:2.12.x Oct 21, 2020
@greg0ire greg0ire deleted the fail-fast branch October 21, 2020 06:51
rgrellmann added a commit to Rossmann-IT/dbal that referenced this pull request Mar 7, 2021
Release [2.12.0](https://github.com/doctrine/dbal/milestone/82)

2.12.0
======

- Total issues resolved: **1**
- Total pull requests resolved: **7**
- Total contributors: **5**

Documentation,Static Analysis
-----------------------------

 - [4376: Configuration should not be internal](doctrine#4376) thanks to @BenMorel

CI
--

 - [4374: Reduce number of build jobs](doctrine#4374) thanks to @greg0ire
 - [4365: Fail on extension / tool installation failure](doctrine#4365) thanks to @greg0ire

Bug,Static Analysis
-------------------

 - [4373: Psalm fails on release commits](doctrine#4373) thanks to @morozov

Documentation,Error Handling
----------------------------

 - [4362: Adds exception thrown by execute() method](doctrine#4362) thanks to @toby-griffiths

CI,PHP
------

 - [4361: Test all extensions with PHP8](doctrine#4361) thanks to @greg0ire

PHP
---

 - [4347: &doctrine#91;2.12&doctrine#93; PHP 8 compatibility](doctrine#4347) thanks to @derrabus

# gpg: Signature made Thu Oct 22 20:29:24 2020
# gpg:                using DSA key 1BEDEE0A820BC30D858F9F0C2C3A645671828132
# gpg: Can't check signature: No public key
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants