Skip to content

Increase dependency options#58

Merged
seregazhuk merged 1 commit into
seregazhuk:masterfrom
mmoreram:feature/increased-tests-and-relaxed-dependencies
Dec 7, 2019
Merged

Increase dependency options#58
seregazhuk merged 1 commit into
seregazhuk:masterfrom
mmoreram:feature/increased-tests-and-relaxed-dependencies

Conversation

@mmoreram
Copy link
Copy Markdown
Contributor

@mmoreram mmoreram commented Dec 5, 2019

  • Improves test document

    • PHP7.4
    • Lowest dependencies
    • Highest dependencies
  • Finder is required by some other libraries, and in lowest version of
    dependencies, the library goes to 2.7. We have forced the library
    to have a minimum dependency of ^4.3

@mmoreram mmoreram force-pushed the feature/increased-tests-and-relaxed-dependencies branch 4 times, most recently from e428884 to 5e2e86c Compare December 5, 2019 13:51
Copy link
Copy Markdown
Owner

@seregazhuk seregazhuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mmoreram Thank you for PR!
Just one little fix and I'll merge it 👍

Comment thread composer.json Outdated
"ext-json": "*",
"ext-pcntl": "*",
"react/stream": "^1.0.0",
"Symfony/finder": "^4.3 || ^5.0",
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It works, but I think it should be lower-case.

- Improves test document
    + PHP7.4
    + Lowest dependencies
    + Highest dependencies

- Finder is required by some other libraries, and in lowest version of
  dependencies, the librarie goes to 2.7. We have forced the library
  to have a minimum dependency of ^4.3
@mmoreram mmoreram force-pushed the feature/increased-tests-and-relaxed-dependencies branch from 5e2e86c to 678f954 Compare December 6, 2019 18:32
@seregazhuk seregazhuk merged commit 4313f3a into seregazhuk:master Dec 7, 2019
@mmoreram mmoreram deleted the feature/increased-tests-and-relaxed-dependencies branch January 3, 2020 09:37
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