Skip to content

Additional integration tests #90

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

Merged
merged 14 commits into from
Jun 29, 2022
Merged

Conversation

boesing
Copy link
Member

@boesing boesing commented Jun 29, 2022

Q A
QA yes

Description

While working on #83, I find more and more tests which should be added before we are merging the typescript refactoring.
This will ensure that we are backwards compatible.

Signed-off-by: Maximilian Bösing <[email protected]>
boesing added 6 commits June 29, 2022 14:41
…iner

We do have to change the `entrypoint` of the docker container to verify that the `diff` is really passed to our GHA within CI. Therefore, we had to add a `shebang` to the `index.js` along with removing the `node` command prefix.

Signed-off-by: Maximilian Bösing <[email protected]>
boesing added 5 commits June 29, 2022 22:26
Since we do use filesystem access to actually verify if the directory/file exist, we do have to create the `docs/book` directory to make the matcher pass.

Signed-off-by: Maximilian Bösing <[email protected]>
…dy target only one PHP version

Signed-off-by: Maximilian Bösing <[email protected]>
@boesing boesing force-pushed the qa/additional-tests branch from fa76427 to e6cfa26 Compare June 29, 2022 20:50
@boesing boesing marked this pull request as ready for review June 29, 2022 23:44
@boesing boesing merged commit 05f7639 into laminas:1.12.x Jun 29, 2022
@boesing boesing deleted the qa/additional-tests branch June 29, 2022 23:44
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