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

PHP 8.1 support #94

Closed
wants to merge 7 commits into from
Closed

Conversation

chris1312
Copy link

PHP 8.1 support

@Ocramius Ocramius added the Enhancement New feature or request label Feb 10, 2022
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.

Needs composer.lock update

@froschdesign froschdesign linked an issue Feb 10, 2022 that may be closed by this pull request
composer.json Outdated Show resolved Hide resolved
composer.json Outdated Show resolved Hide resolved
chris1312 and others added 5 commits February 24, 2022 11:59
Signed-off-by: kristijonas bulzgis <[email protected]>
Co-authored-by: Piotr Zygmuntowicz <[email protected]>
Signed-off-by: kristijonas bulzgis <[email protected]>
Signed-off-by: kristijonas bulzgis <[email protected]>
Signed-off-by: kristijonas bulzgis <[email protected]>
Co-authored-by: Frank Brückner <[email protected]>
Signed-off-by: kristijonas bulzgis <[email protected]>
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.

Seems like CI is in shambles here 😬

@chris1312
Copy link
Author

So, for PHP 8.1, the current obstacles are upgrades of these modules:

  • laminas-api-tools/api-tools-mvc-auth
  • laminas-api-tools/api-tools-oauth2

Also, I suggest dropping support for PHP 7.3, and avoid committing composer.lock to the repository (composer.lock can be generated automatically during the CI run).

@Ocramius
Copy link
Member

avoid committing composer.lock to the repository

We will commit composer.lock to all our repositories, to guarantee reproducible builds.

We are currently adding @renovatebot to all our repos to keep it fresh on a daily basis, and eagerly discover incompatibilities.

Totally okay with dropping php 7.3: we don't support EOL php versions anyway

Signed-off-by: kristijonas bulzgis <[email protected]>
composer.lock Outdated Show resolved Hide resolved
Signed-off-by: kristijonas bulzgis <[email protected]>
@Ocramius Ocramius mentioned this pull request Mar 25, 2022
@Ocramius
Copy link
Member

Ocramius commented Jun 6, 2022

Moved to #95 meanwhile, thanks for all your help though, still adding this to release notes for accrediting it :-)

@Ocramius Ocramius self-assigned this Jun 6, 2022
@Ocramius Ocramius closed this Jun 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PHP 8.1 support
4 participants