Skip to content

Add tests for the details urls #2922

Merged
merged 2 commits into from Mar 9, 2014

2 participants

@FichteFoll
Collaborator

I decided to allow trailing slashes for now because they seem to work
internally (at least thos package seem to be installable) and quite a
few use it.

For review and visibility @joneshf and @wbond.

FichteFoll added some commits Mar 8, 2014
@FichteFoll FichteFoll Unify details urls
Use those which are used internally (manly for FalconSublimeTools).
cf924e5
@FichteFoll FichteFoll Add tests for the details urls
I decided to allow trailing slashes for now because they seem to work
internally (at least thos package seem to be installable) and quite a
few use it.
ca8f534
@joneshf
Collaborator
joneshf commented Mar 9, 2014

Looks good tome.

@FichteFoll FichteFoll merged commit 7736fec into master Mar 9, 2014

1 check passed

Details default The Travis CI build passed
@FichteFoll FichteFoll deleted the test_details branch Jul 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.