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

Documentation: Clarify that computed properties are lazy. #182

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

Kelketek
Copy link
Contributor

Small documentation change-- clarifies that computed properties are lazily evaluated. Resolves #178.

Copy link
Collaborator

@theengineear theengineear left a comment

Choose a reason for hiding this comment

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

Thanks for the update @Kelketek — makes sense to call that out explicitly 👌

@klebba
Copy link
Collaborator

klebba commented Jul 23, 2024

+1 — thanks @Kelketek — out of curiosity could you share how you came across this project/what you are using it for? Would love to hear more — thanks!

@Kelketek
Copy link
Contributor Author

@klebba I just started browsing Netflix's frontend repositories, curious what you all were working on. I found this interesting since previous to this point I've been using tools like React and Vue, and I hadn't seen much in terms of serious use of Web Components.

Diving into this project was a good way to learn about Web Components-- I'd not used them before. I'm hoping to have a PR for you soon to resolve #179 as well, or at least greatly mitigate it between a code tweak and additional documentation. I should hopefully have that for you by tomorrow.

@klebba
Copy link
Collaborator

klebba commented Jul 26, 2024

Thanks for the insight @Kelketek and it's nice to hear that this project was helpful for you!

@theengineear theengineear merged commit e60005b into Netflix:main Jul 26, 2024
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.

Lazily computed properties can cause confusion
3 participants