Skip to content
This repository has been archived by the owner on Jun 30, 2020. It is now read-only.

Update fast-route dependency #53

Merged

Conversation

sander-bol
Copy link
Contributor

Update dependency on nikic/fast-route from "0.*" to "^0.7" to resolve conflict with "league/route".

@sander-bol
Copy link
Contributor Author

@oscarotero Could you merge this one? This broken dependency is breaking all builds currently, as mentioned in #46

@abacaphiliac
Copy link
Contributor

+1

@oscarotero oscarotero merged commit d9725e9 into oscarotero:master Nov 15, 2016
@oscarotero
Copy link
Owner

Merged!. Thank you. I'm curious about the reason why league/route does not use the latest version of fast-route. The version 2.0 has been released yesterday and it still using the 0.7 version.

@abacaphiliac
Copy link
Contributor

@oscarotero let's find out together!
thephpleague/route#135

seems that league's route works with lots of fast-route versions including the most recent tags. i added highest and lowest dependency version builds. looks like the package has pretty good coverage, so we'll see what they say : )

@abacaphiliac
Copy link
Contributor

@oscarotero my change to thephpleague/route was accepted. we might be able to loosen the dependency range now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants