Add setState example to Basic Prop Types

[WillowP]

Solves #574

[swyxio]

1 suggesiton but lgtm, letting @filiptammergard review

[filiptammergard]

Sweet! Small suggestion, let's change to that and we're ready to merge!

[filiptammergard]

@WillowP Last thing, can you run yarn gen-readme and push the diff?

[WillowP]

Sweet! Small suggestion, let's change to that and we're ready to merge!

On second thought, I wonder if taking out the reference to useState in the comment might be a bad idea. Technically there are different kinds of state setting functions (ie the dispatch returned by useReducer), but this is specifically for the one returned by useState. Readers might be confused if we just call it "the state setter function".

What about:

the state setter function returned by `useState`

[filiptammergard]

Sweet! Small suggestion, let's change to that and we're ready to merge!

On second thought, I wonder if taking out the reference to useState in the comment might be a bad idea. Technically there are different kinds of state setting functions (ie the dispatch returned by useReducer), but this is specifically for the one returned by useState. Readers might be confused if we just call it "the state setter function".

What about:

the state setter function returned by `useState`

Good point, yes let's go for that tweak!

[filiptammergard]

Thanks @WillowP!