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

Update workflow #1

Closed
wants to merge 1 commit into from
Closed

Update workflow #1

wants to merge 1 commit into from

Conversation

abenevaut
Copy link

@abenevaut abenevaut commented Sep 15, 2023

@abenevaut abenevaut marked this pull request as ready for review September 15, 2023 19:41
@abenevaut abenevaut changed the base branch from master to issue-php82 September 15, 2023 19:42
@abenevaut abenevaut marked this pull request as draft September 15, 2023 19:42
@abenevaut abenevaut changed the title Issue php82 ci Update workflow Sep 15, 2023
@abenevaut abenevaut marked this pull request as ready for review September 15, 2023 19:47
@remicollet
Copy link
Owner

Not the right place for PR, use https://github.com/laruence/yaf

@remicollet remicollet closed this Sep 16, 2023
@abenevaut
Copy link
Author

You propose that PR laruence#579 that upgrade yaf to be handled by php 8.2 & php 8.3 without upgrading the CI pipeline.

The current PR is proposed to complete your partial PR and upgrade CI pipeline to test your work. I guess this is the correct place to mix our work to upgrade yaf to php 8.3 with your awareness on PHP and my awareness on CI to laruence repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants