Skip to content

Conversation

@ericproulx
Copy link
Contributor

This PR apply the same strategy as Rails when normalizing path.

@ericproulx ericproulx marked this pull request as ready for review February 16, 2025 13:18
@ericproulx ericproulx requested a review from dblock February 16, 2025 13:18
@grape-bot
Copy link

grape-bot commented Feb 16, 2025

1 Warning
⚠️ There're library changes, but not tests. That's OK as long as you're refactoring existing code.

Generated by 🚫 Danger

Copy link
Member

@dblock dblock left a comment

Choose a reason for hiding this comment

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

In what situations does this change behavior? There's a lot more going on here than in the Rails commit :) At the very least I feel like we need to bring in some tests from Rails for normalize_path if we're going to be doing it here as well.

@ericproulx ericproulx requested a review from dblock February 19, 2025 12:16
@ericproulx
Copy link
Contributor Author

@dblock I've added some specs. Basically, it does exactly the same thing but there's a faster path for the usual cases.

@dblock dblock merged commit 6e6958f into ruby-grape:master Feb 19, 2025
63 checks passed
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.

3 participants