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

Reuse workflows #373

Merged
merged 3 commits into from
Oct 17, 2021
Merged

Reuse workflows #373

merged 3 commits into from
Oct 17, 2021

Conversation

greg0ire
Copy link
Member

No description provided.

@greg0ire greg0ire force-pushed the reuse-workflows branch 2 times, most recently from f337f57 to 2f12c68 Compare October 16, 2021 17:15
Looks like we are hitting vimeo/psalm#5570
@greg0ire greg0ire added this to the 1.5.2 milestone Oct 16, 2021
# https://github.com/doctrine/.github/issues/3
- name: "Run PHP_CodeSniffer"
run: "vendor/bin/phpcs -q --no-colors --report=checkstyle | cs2pr"
uses: "doctrine/.github/.github/workflows/coding-standards.yml@1.1.1"
Copy link
Member

Choose a reason for hiding this comment

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

You're switching to PHP 8.0 and I don't see if this was done on purpose.

Copy link
Member Author

Choose a reason for hiding this comment

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

It's on purpose, I don't think the version of PHP matters for phpcs

@greg0ire greg0ire merged commit 60eb26d into doctrine:1.5.x Oct 17, 2021
@greg0ire greg0ire deleted the reuse-workflows branch October 17, 2021 21:52
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.

2 participants