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

Resolve the env vars for conventions #760

Closed

Conversation

janvernieuwe
Copy link
Collaborator

Q A
Branch master
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Documented? no
Fixed tickets Fixes #755

See the issue

@veewee
Copy link
Contributor

veewee commented Apr 24, 2020

Can you add an E2E test for this to make sure it works as expected?

ashokadewit pushed a commit to mediact/testing-suite that referenced this pull request May 29, 2020
This fixes:

```
The option "strict" with value "%composer.strict%" is expected to be of
type "bool", but is of type "string".
```

A pull request exists on phpro/grumphp#760 which
also fixes this issue. When that pull request has been merged this
extension can be removed.
@janvernieuwe
Copy link
Collaborator Author

This has been tested in #741 as well, and seems to be resolved there. Conventions will be usable again when this PR is released.

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.

Convention variables validation bug
2 participants