Skip to content

Conversation

@RobbieTheWagner
Copy link
Contributor

No description provided.

@RobbieTheWagner
Copy link
Contributor Author

@vladislavarsenev @byara this is finally the PR I really needed so we can support Ember's gjs/gts formats. Please let me know what you think of the changes! I also fixed the examples more, as they needed the svelte and ember-template-tag plugins to run.

@vladislavarsenev
Copy link
Collaborator

Thank you for your hard work on this! LTGM

};
}

export default class Foo extends Component<FooSignature> {} No newline at end of file
Copy link

Choose a reason for hiding this comment

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

Would it make sense to include a <template> block in these examples?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Good idea! Hey @RobbieTheWagner could you please some examples with <template> block?

Copy link
Collaborator

@byara byara left a comment

Choose a reason for hiding this comment

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

Looks good to me. I left a comment only about additional test cases.
Please do not forget to add the support for gjs/gts in the readme file.
Thank you so much for adding this feature 🚀

Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe in another test case we can have something with the <template> block here too?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Turns out things are not working when you have a <template> 😅. I will work on a fix.

@RobbieTheWagner
Copy link
Contributor Author

Closing in favor of #377

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