Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

docs: fix typos #1490

Merged
merged 3 commits into from
Mar 7, 2022
Merged

docs: fix typos #1490

merged 3 commits into from
Mar 7, 2022

Conversation

muziejus
Copy link
Contributor

@muziejus muziejus commented Mar 4, 2022

Currently in the docs, @attr() declare name? is typed as string, and no example of
@attr('string') is given. I'm not sure if showing in the documentation
that not passing a transform to @attr() should explicitly default to
string is the correct thing to do, but I definitely think we should
model the use of @attr('string') in the code examples.

Currently in the docs, `@attr() declare name?` is typed as `string`, and no example of
`@attr('string')` is given. I'm not sure if showing in the documentation
that not passing a transform to `@attr()` should explicitly default to
string is the correct thing to do, but I definitely think we should
model the use of `@attr('string')` in the code examples.
Copy link
Member

@chriskrycho chriskrycho left a comment

Choose a reason for hiding this comment

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

Good catches. I'd like to hear from Data folks on the @attr() default, so once you split that out, let me know and I'll merge the typo fixes. Thanks!

@muziejus muziejus changed the title docs: fix typos and replace use of attr() docs: fix typos Mar 6, 2022
@chriskrycho chriskrycho merged commit d2da87a into typed-ember:master Mar 7, 2022
@chriskrycho
Copy link
Member

Thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants