Skip to content
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

Allow outputting URLs with trailing slash #692

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Conversation

bakerkretzmar
Copy link
Collaborator

Adds a trailing_slash config option that makes ->getUrl() always output a URL with a trailing slash. Doesn't affect ->getPath() or anything else.

Fixes #679 I think?

....✓...................✓........✓.............✓....✓..........✓✓......✓..✓.
......✓✓✓✓✓.✓..✓....✓✓....✓............

──────────────────────────────────────────────────────────────────── Laravel
FIXED ................................. 115 files, 19 style issues fixed
✓ src/Collection/Collection.php new_with_parentheses, unary_operator_spaces…
✓ src/Console/BuildCommand.php new_with_parentheses, unary_operator_spaces,…
✓ src/Events/EventBus.php new_with_parentheses
✓ src/Exceptions/Handler.php nullable_type_declaration_for_default_null_val…
✓ src/File/TemporaryFilesystem.php new_with_parentheses
✓ src/Scaffold/PresetPackage.php new_with_parentheses, unary_operator_space…
✓ src/SiteData.php new_with_parentheses
✓ src/Support/helpers.php new_with_parentheses, unary_operator_spaces, not_…
✓ tests/CustomCommandTest.php new_with_parentheses
✓ tests/CustomScaffoldInstallerTest.php new_with_parentheses
✓ tests/DefaultScaffoldInstallerTest.php new_with_parentheses
✓ tests/DotInFileNameTest.php new_with_parentheses
✓ tests/FilePathTest.php new_with_parentheses
✓ tests/InitCommandTest.php new_with_parentheses
✓ tests/PhpOpenTagInMarkdownTest.php new_with_parentheses
✓ tests/PresetScaffoldBuilderTest.php new_with_parentheses
✓ tests/SnapshotsTest.php new_with_parentheses, php_unit_method_casing, una…
✓ tests/TestCase.php new_with_parentheses, class_definition, braces_positio…
✓ tests/snapshots/default-trailing-slash/config.php list_syntax
)
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.

Allow formatting of pretty URL with trailing slash
3 participants