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

Introduce roave/backward-compatibility-check and @default php version #158

Merged
merged 6 commits into from
Jul 11, 2023

Conversation

boesing
Copy link
Member

@boesing boesing commented Jul 8, 2023

Q A
Documentation yes
New Feature yes

Description

This adds roave-backward-compatibility-check to the globally available tools.
By moving setup/staabm-annotate-pull-request-from-checkstyle into the roots composer.json, we allow renovate to automatically bump composer dependencies.

By adding another GHA, we do also verify that the tools we are installing are actually executable (i.e. due to the aggressive PHP version requirements of roave packages).

@default for php job configuration will be provided by laminas/laminas-ci-matrix-action#126 once my changes were pushed (not yet possible due to PR/fork limitations).

@boesing boesing added this to the 1.33.0 milestone Jul 8, 2023
@boesing boesing requested a review from Ocramius July 8, 2023 00:27
Dockerfile Outdated Show resolved Hide resolved
@boesing boesing force-pushed the feature/roave-backward-compatibility-check branch from 9fdd837 to 925297a Compare July 8, 2023 20:22
Dockerfile Show resolved Hide resolved
@boesing
Copy link
Member Author

boesing commented Jul 8, 2023

Ready for review.

Dockerfile Show resolved Hide resolved
…vailable tool

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
@boesing boesing force-pushed the feature/roave-backward-compatibility-check branch from 925297a to 97ec86f Compare July 8, 2023 20:45
entrypoint.sh Show resolved Hide resolved
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
@boesing boesing changed the title Introduce roave/backward-compatibility-check Introduce roave/backward-compatibility-check and @default php version Jul 8, 2023
Dockerfile Show resolved Hide resolved
Dockerfile Show resolved Hide resolved
Dockerfile Show resolved Hide resolved
entrypoint.sh Show resolved Hide resolved
Copy link
Member

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

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

Just making the LGTM official :D

Great work, @boesing!

@boesing boesing merged commit de5ce6e into 1.33.x Jul 11, 2023
4 checks passed
@boesing boesing deleted the feature/roave-backward-compatibility-check branch July 11, 2023 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants