Skip to content

Add support for laminas/laminas-diactoros v2.#13

Merged
Naktibalda merged 1 commit intoCodeception:masterfrom
samuelnogueira:diactoros_v2
Dec 26, 2021
Merged

Add support for laminas/laminas-diactoros v2.#13
Naktibalda merged 1 commit intoCodeception:masterfrom
samuelnogueira:diactoros_v2

Conversation

@samuelnogueira
Copy link
Copy Markdown
Contributor

No description provided.

* Main workflow now tests both 1.8 and 2.0 versions of laminas diactoros for each PHP version.
* Use `update` instead of `install` to install composer dependencies to allow `--with` param. This change will also address the following warning in workflow, since composer.lock file is not in version control:
  > No composer.lock file present. Updating dependencies to latest instead of installing from lock file. See https://getcomposer.org/install for more information.
@samuelnogueira
Copy link
Copy Markdown
Contributor Author

Please confirm if changing the github action workflow to validate compatibility with both diactoros major versions is acceptable.
I couldn't find examples of similar split testing within Codeception except for these:

@Naktibalda
Copy link
Copy Markdown
Member

Looks good to me.

@Naktibalda Naktibalda merged commit e732900 into Codeception:master Dec 26, 2021
@samuelnogueira samuelnogueira deleted the diactoros_v2 branch December 26, 2021 17:48
@Naktibalda
Copy link
Copy Markdown
Member

Released as 3.0.2

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