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

Add PHP 8 compatibility and Use github actions as test CI #355

Merged

Conversation

alexander-schranz
Copy link
Contributor

@alexander-schranz alexander-schranz commented Jan 16, 2021

See alexander-schranz#1 for Github action run.

@alexander-schranz alexander-schranz force-pushed the feature/add-gitlab-actions branch 2 times, most recently from df45672 to c82f8b7 Compare January 16, 2021 12:45
@alexander-schranz alexander-schranz force-pushed the feature/add-gitlab-actions branch 4 times, most recently from b08fa3e to f6e8281 Compare January 16, 2021 13:26
@greg0ire
Copy link
Member

Hi! Nice contribution! Can you please make it closer to our template? https://github.com/doctrine/.github/blob/main/workflow-templates/continuous-integration.yml (most notably by using ramsey/composer-install)

@alexander-schranz alexander-schranz force-pushed the feature/add-gitlab-actions branch 4 times, most recently from c1b5d70 to 4ea2d89 Compare January 16, 2021 14:14
@alexander-schranz
Copy link
Contributor Author

@greg0ire thx for the feedback did change that and tests are now green. Github will only show them here when this is merged so state of the PR CI can be seen here: alexander-schranz#1

@alexander-schranz alexander-schranz mentioned this pull request Jan 16, 2021
52 tasks
.github/workflows/test-application.yaml Outdated Show resolved Hide resolved
.github/workflows/test-application.yaml Outdated Show resolved Hide resolved
.github/workflows/test-application.yaml Outdated Show resolved Hide resolved
.github/workflows/test-application.yaml Outdated Show resolved Hide resolved
.github/workflows/test-application.yaml Outdated Show resolved Hide resolved
.github/workflows/test-application.yaml Outdated Show resolved Hide resolved
.github/workflows/test-application.yaml Outdated Show resolved Hide resolved
@greg0ire
Copy link
Member

tests are now green

Looks like they no longer are green

@alexander-schranz alexander-schranz force-pushed the feature/add-gitlab-actions branch from 1f81aae to aa10ed6 Compare January 16, 2021 17:35
@alexander-schranz alexander-schranz mentioned this pull request Jan 16, 2021
@alexander-schranz alexander-schranz force-pushed the feature/add-gitlab-actions branch 2 times, most recently from 0d77482 to b26bb8c Compare January 16, 2021 17:45
@alexander-schranz
Copy link
Contributor Author

@greg0ire green again ;)

@greg0ire greg0ire force-pushed the feature/add-gitlab-actions branch from b26bb8c to 303c0ab Compare January 16, 2021 18:20
greg0ire and others added 2 commits January 16, 2021 19:21
Co-Authored-By: Alexander Schranz <alexander@sulu.io>
Co-Authored-By: Alexander Schranz <alexander@sulu.io>
@greg0ire greg0ire force-pushed the feature/add-gitlab-actions branch from 303c0ab to 45a9d55 Compare January 16, 2021 18:22
@greg0ire greg0ire merged commit 0f6b74e into doctrine:master Jan 16, 2021
@greg0ire
Copy link
Member

Thanks a lot @alexander-schranz !

@alexander-schranz
Copy link
Contributor Author

@greg0ire thx for the quick review! Is it possible to release this soon., so I can check if the symfony-cmf packages work as expected.

@alexander-schranz
Copy link
Contributor Author

alexander-schranz commented Jan 16, 2021

@greg0ire Thx for tagging 🙏 !

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