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

use proper example code tag in hooks JSDoc comments #1257

Merged
merged 3 commits into from
Apr 23, 2019

Conversation

MrWolfZ
Copy link
Contributor

@MrWolfZ MrWolfZ commented Apr 23, 2019

While working on a PR for the TypeScript types the linter use there made me realize I used an invalid syntax for jsdoc example code. This PR fixes the inline comments to use @example. It also fixes some other minor typos and mistakes in the example code.

@netlify
Copy link

netlify bot commented Apr 23, 2019

Deploy preview for react-redux-docs ready!

Built with commit 53acb63

https://deploy-preview-1257--react-redux-docs.netlify.com

@markerikson markerikson merged commit 92baa04 into reduxjs:v7-hooks-alpha Apr 23, 2019
timdorr pushed a commit that referenced this pull request May 30, 2019
* use proper example code tag in hooks JSDoc comments

* fix mistake in `useActions` hook example code

* remove TypeScript annotations from example code and adjust `useReduxContext` hook to also use @example JSDoc tag
timdorr pushed a commit that referenced this pull request Jun 11, 2019
* use proper example code tag in hooks JSDoc comments

* fix mistake in `useActions` hook example code

* remove TypeScript annotations from example code and adjust `useReduxContext` hook to also use @example JSDoc tag
albertodev7 pushed a commit to albertodev7/react-redux that referenced this pull request Dec 8, 2022
* use proper example code tag in hooks JSDoc comments

* fix mistake in `useActions` hook example code

* remove TypeScript annotations from example code and adjust `useReduxContext` hook to also use @example JSDoc tag
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.

2 participants