Skip to content

Conversation

@wout
Copy link
Contributor

@wout wout commented Nov 15, 2025

Purpose

Adds a compile-time error in case a path alias is not prefixed with a /.

Description

I got bitten by this today. Main routes and prefixes already do this check but not the aliases. I also fixed the punctuation on the other checks because they would render like:

 Error: Path must start with a slash. Example: '/"about"'

Now it's:

 Error: Path must start with a slash. Example: "/about"

Checklist

  • - An issue already exists detailing the issue/or feature request that this PR fixes
  • - All specs are formatted with crystal tool format spec src
  • - Inline documentation has been added and/or updated
  • - Lucky builds on docker with ./script/setup
  • - All builds and specs pass on docker with ./script/test

@jwoertink jwoertink merged commit 669c3f1 into luckyframework:main Nov 17, 2025
7 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.

2 participants