Skip to content

WIP try rajasegar/ember-cli-update-codemods-manifest/ember-angle-brackets-codemod - #606

Closed
kellyselden wants to merge 1 commit into
masterfrom
rajasegar/ember-angle-brackets-codemod
Closed

WIP try rajasegar/ember-cli-update-codemods-manifest/ember-angle-brackets-codemod#606
kellyselden wants to merge 1 commit into
masterfrom
rajasegar/ember-angle-brackets-codemod

Conversation

@kellyselden

Copy link
Copy Markdown
Member

@kellyselden
kellyseldenforce-pushed the rajasegar/ember-angle-brackets-codemod branch 6 times, most recently from 227f603 to b81af95CompareJuly 27, 2019 17:14
@kellyseldenkellyselden changed the title try rajasegar/ember-cli-update-codemods-manifest/ember-angle-brackets-codemodWIP try rajasegar/ember-cli-update-codemods-manifest/ember-angle-brackets-codemodJul 28, 2019
@kellyselden
kellyseldenforce-pushed the rajasegar/ember-angle-brackets-codemod branch from f78da75 to 664598bCompareAugust 17, 2019 09:15
@kellyselden
kellyseldenforce-pushed the rajasegar/ember-angle-brackets-codemod branch from 664598b to 3edc156CompareOctober 10, 2019 10:54
@kellyselden

Copy link
Copy Markdown
MemberAuthor

@kellyselden

Copy link
Copy Markdown
MemberAuthor
 + expected - actual
"styles": {
"app.css": ""
}
"templates": {
- "application.hbs": "{{!-- The following component displays Ember's default welcome message. --}}\r\n<WelcomePage />\r\n{{!-- Feel free to remove this! --}}\r\n\r\n{{outlet}}\r\n\r\n<Ui::Button @text=\"Click me\" />"
+ "application.hbs": "{{! The following component displays Ember's default welcome message. }}\r\n<WelcomePage />\r\n{{! Feel free to remove this! }}\r\n{{outlet}}\r\n<Ui::Button @text=\"Click me\" />"
"components": {
".gitkeep": ""
}
}

@rwjbluerwjblue left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@kellyselden - I actually think this PR has the expected wrong, in that last diff you pasted just above it shows that the line endings are fine (AFAICT).

@@ -1,5 +1,5 @@
{{!-- The following component displays Ember's default welcome message. --}}
{{! The following component displays Ember's default welcome message. }}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why are the -- removed here? The original (assuming that test/fixtures/codemod/local/my-app/app/templates/application.hbs represents the input) uses {{!--, so I would expect the output to continue to use them.

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

These were remnants from when the codemod did a lot more formatting, and the expected was altered to match. This has been reverted now.


{{outlet}} No newline at end of file
{{! Feel free to remove this! }}
{{outlet}}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There should be a newline both just before and just after {{outlet}} (based on the input template in test/fixtures/codemod/local/my-app/app/templates/application.hbs)

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

These were remnants from when the codemod did a lot more formatting, and the expected was altered to match. This has been reverted now.

@kellyselden
kellyseldenforce-pushed the rajasegar/ember-angle-brackets-codemod branch 14 times, most recently from 8ad3625 to 379ce73CompareNovember 15, 2019 13:54
@kellyselden
kellyseldenforce-pushed the rajasegar/ember-angle-brackets-codemod branch from 379ce73 to d014463CompareNovember 15, 2019 15:32
kellyselden pushed a commit to rajasegar/ember-cli-update-codemods-manifest that referenced this pull request Nov 17, 2019
@kellyselden

Copy link
Copy Markdown
MemberAuthor

@kellyselden
kellyselden deleted the rajasegar/ember-angle-brackets-codemod branch November 17, 2019 21:41
@kellyselden

Copy link
Copy Markdown
MemberAuthor

Line endings used to be a problem, but aren't anymore. Thanks to everyone involved!

kellyselden pushed a commit to ember-cli/ember-cli-update-codemods-manifest that referenced this pull request Nov 17, 2019
Sign up for freeto 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

@kellyselden@rwjblue