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.

Sign up for GitHub

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

Jump to bottom

Add docs for useCallback #586

Merged
merged 12 commits into from
Nov 2, 2022
Merged

Add docs for useCallback #586

merged 12 commits into from
Nov 2, 2022

Conversation

hinsxd
Copy link
Contributor

@hinsxd hinsxd commented Oct 30, 2022 •
edited
Loading

Addresses useCallback implicit any in #495

eps1lon
eps1lon reviewed Oct 31, 2022
View reviewed changes
README.md Outdated Show resolved Hide resolved
eps1lon
eps1lon reviewed Oct 31, 2022
View reviewed changes
README.md Outdated Show resolved Hide resolved
eps1lon
eps1lon reviewed Oct 31, 2022
View reviewed changes
README.md Outdated Show resolved Hide resolved
eps1lon
eps1lon reviewed Oct 31, 2022
View reviewed changes
README.md Show resolved Hide resolved
eps1lon
eps1lon reviewed Oct 31, 2022
View reviewed changes
README.md Outdated Show resolved Hide resolved
eps1lon
eps1lon approved these changes Oct 31, 2022
View reviewed changes
Copy link
Member

@eps1lon eps1lon left a comment

Choose a reason for hiding this comment

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

Thanks! Polished some wording.

@hinsxd
Copy link
Contributor Author

hinsxd commented Oct 31, 2022

Thanks for the edit!

filiptammergard
filiptammergard approved these changes Nov 1, 2022
View reviewed changes
Copy link
Collaborator

@filiptammergard filiptammergard left a comment

Choose a reason for hiding this comment

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

A couple of suggestions from my side as well, otherwise looks good!

README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
hinsxd and others added 3 commits November 2, 2022 09:46
@hinsxd @filiptammergard
Update README.md
4a9964d 
Co-authored-by: Filip Tammergård <[email protected]>
@hinsxd @filiptammergard
Update README.md
e2eef08 
Co-authored-by: Filip Tammergård <[email protected]>
@hinsxd
chore: update
4d19e85
@eps1lon eps1lon merged commit c171740 into typescript-cheatsheets:main Nov 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@filiptammergard filiptammergard filiptammergard approved these changes

@eps1lon eps1lon eps1lon approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hinsxd @eps1lon @filiptammergard