Unused var + linting

[goldylucks]

Isn't passing value enough?

Here:
https://github.com/ReactTraining/react-workshop/blob/13748e43467581c2933dbd878f1cc1aef6a08263/subjects/07-Compound-Components/solution.js#L62-L66

Can do this instead:

  select(value) {
    this.setState({ value }, () => {
      this.props.onChange(value); // no need to do this.state.value
    });
  }

Also having eslint installed would have flagged that value as an unused param.

[mjackson]

You can do either one. this.state.value makes it clear that we're grabbing the value from this.state after it has been updated.

The value param is used in the setState call, so it's not unused.