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

Fix passthrough rule that may duplicate extension when routing items #251

Merged
merged 1 commit into from Apr 23, 2013

Conversation

Projects
None yet
2 participants
@gpakosz
Member

gpakosz commented Feb 25, 2013

This pull request implements a possible fix for #243

@ddfreyne

This comment has been minimized.

Show comment
Hide comment
@ddfreyne

ddfreyne Feb 25, 2013

Member

(For future reference: this is pull request #246 re-created.)

Member

ddfreyne commented Feb 25, 2013

(For future reference: this is pull request #246 re-created.)

@ddfreyne

This comment has been minimized.

Show comment
Hide comment
@ddfreyne

ddfreyne Mar 31, 2013

Member

@gpakosz Can you provide a test case for this? Once the test case is done, I can merge it into 3.6.x.

Member

ddfreyne commented Mar 31, 2013

@gpakosz Can you provide a test case for this? Once the test case is done, I can merge it into 3.6.x.

@ddfreyne

This comment has been minimized.

Show comment
Hide comment
@ddfreyne

ddfreyne Apr 15, 2013

Member

I'll write the tests for this (not trivial).

Member

ddfreyne commented Apr 15, 2013

I'll write the tests for this (not trivial).

@ddfreyne ddfreyne merged commit 6352bce into nanoc:release-3.6.x Apr 23, 2013

1 check failed

default The Travis build failed
Details
@ddfreyne

This comment has been minimized.

Show comment
Hide comment
@ddfreyne

ddfreyne Apr 23, 2013

Member

Fixed in b252f49. Thanks!

Member

ddfreyne commented Apr 23, 2013

Fixed in b252f49. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment