Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 08:49
    meowtec commented #2239
  • Oct 20 10:51
    golopot opened #2469
  • Oct 20 10:31
    golopot opened #2468
  • Oct 20 08:11
    rhalaly edited #2467
  • Oct 20 08:09
    rhalaly edited #2467
  • Oct 20 08:09
    rhalaly opened #2467
  • Oct 19 22:02
    ljharb commented #613
  • Oct 19 20:45
    EECOLOR commented #613
  • Oct 19 19:29
    ljharb labeled #613
  • Oct 19 19:28
    ljharb closed #613
  • Oct 19 19:28
    ljharb commented #613
  • Oct 19 19:26
    EECOLOR commented #613
  • Oct 18 23:42
    ljharb labeled #2466
  • Oct 18 23:42
    ljharb labeled #2466
  • Oct 18 23:42
    ljharb commented #2466
  • Oct 18 21:49
    Lucasdsk opened #2466
  • Oct 18 18:38
    derekjobst commented #2344
  • Oct 18 14:01
    krawaller commented #2073
  • Oct 17 06:36
    SerdarMustafa1 opened #2465
  • Oct 17 00:48
    JHilker commented #2457
Berkeley Martinez
@BerkeleyTrue
Woot!
Mathieu M-Gosselin
@mathieumg
I didn't know this room existed, might want to add the Gitter badge! :P
Mathieu M-Gosselin
@mathieumg
@yannickcr Did you have a release in mind soon? I don't mean to press, but it's been a couple of weeks and there have been two eslint releases since the last eslint-plugin-react minor release. :)
Mathieu M-Gosselin
@mathieumg
Well, that answers it! :tada:
Mathieu M-Gosselin
@mathieumg
ok last message here is mine 2 months ago, is anyone using this room? :P
I was wondering how https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-indent-props.md behaves for single-line components (with one or no prop)
webmato
@webmato

hello to all.
I have this issue with ESLint and React props / propTypes

...
this.props.hasOwnProperty('modelConfig');   /* =>  ESLint error: "no-prototype-builtins"   */
this.props.hasOwnProperty.call('modelConfig');   /*  => ESLint error: "react/prop-types"  (underlined is the "call" word) */
...
MyClass.propTypes = {
  modelConfig: ImmutablePropTypes.map,
};

How to solve it?

webmato
@webmato
edit: I have typo, instead this.props.hasOwnProperty.call('modelConfig') should be this.props.hasOwnProperty.call(this.props, 'modelConfig') , but still the same ESLint issue.
webmato
@webmato
good bye hasOwnProperty - so easy solution with ES6 :clap:
const { modelConfig } = this.props;
if(modelConfig) { ... }
Joachim Seminck
@jseminck

Hi - so in one of the PRs it was mentioned that the code base should ideally only contain const and let, and no more var. So would you be interested in a PR for this?

I don't mind to do it as it would provide me a good opportunity to go through the code, but it would still take a couple of days as I want to take my time and read the code rather than just skim through it and just do find/replace.

Joachim Seminck
@jseminck
@yannickcr any chance to get a new version released? Many people have asked because of yannickcr/eslint-plugin-react#1290 -- it gives an annoying warning in the IDE when using ESLint 4.2.0
Erwann Mest
@kud
Hello
is there someone here? :D