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

[ui] Trim variable path before save #18198

Merged
merged 1 commit into from
Aug 17, 2023

Conversation

philrenaud
Copy link
Contributor

@philrenaud philrenaud commented Aug 14, 2023

Resolves #18197

We previously had a trimPath helper for making sure that, if our paths ended/started in / it would be removed. This does the same for basic whitespace using JS's string trim

@github-actions
Copy link

Ember Test Audit comparison

main 80fa2c520ac7795044af8e6d64b37737c3482c33 change
passes 1506 1506 0
failures 1 1 0
flaky 0 0 0
duration 000ms 000ms -000ms

Copy link
Contributor

@angrycub angrycub left a comment

Choose a reason for hiding this comment

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

Nice!

@philrenaud philrenaud added the backport/1.6.x backport to 1.6.x release line label Aug 17, 2023
@philrenaud philrenaud force-pushed the 18197-ui-trim-variable-path-names-on-save branch from 80fa2c5 to 13b4d98 Compare August 17, 2023 14:45
@philrenaud philrenaud merged commit ba7892d into main Aug 17, 2023
@philrenaud philrenaud deleted the 18197-ui-trim-variable-path-names-on-save branch August 17, 2023 14:46
@philrenaud philrenaud restored the 18197-ui-trim-variable-path-names-on-save branch August 23, 2023 15:24
@philrenaud philrenaud deleted the 18197-ui-trim-variable-path-names-on-save branch August 23, 2023 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.6.x backport to 1.6.x release line
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ui] Trim variable path names on save
2 participants