Hi team! I was wanted to kindly ask the maintainers to have a look, if possible. I have raised the MR 15 days ago, I was hoping for some feedback :) Thank you!

Any updates on this? Getting some annoying linter errors in the IDE because of #60 :)

Any updates on this? Getting some annoying linter errors in the IDE because of #60 :)

There is not a single thing I can do, unfortunately :(

Yeah, sorry. Wasn't aiming at you, specifically. Just wanted to bump this one for the powers that be to get notified. ;)

Hi @cezudas, I tried out your branch and while it is working with eslint8, it seems to be causing a bug

Specifically, it seems that it incorrectly marks props/methods decorated with any decorator (@Prop, @Method, @Listen, @Watch, @Event, @Element) as needing to have the private keyword added, which was not required before and actually conflicts with other rules @stencil/props-must-be-public and @stencil/methods-must-be-public

I don't understand what's happening, because if I run the tests in your repo, all the rules are passing and I can see the tests do cover the scenario I described above...

Could this be affected by how typescript compiles?

I think it's time to give a second try on this PR.
I created a branch where I updated all the dev dependencies and it seems to work fine.
I also created a build to test in my project and didn't see any new issues.

Do you guys need help to get this done?

@raphaelpor can you share the branch? the current branch is definitely causing issues for me (see my comment above) so it would be good to try out your branch too

@theo-staizen False alarm. I checked again my changes and I do get the same issues as you.
I will try to continue working on that to understand what's going on and see if I can come up with a solution.

Sorry for that.

Hello,

I was playing around with this and discovered that if I updated as follows:
@types/eslint: "8.4.10!
eslint: "8.31.0"
@typescript-eslint/parser: 5.48.1
@typescript-eslint/eslint-plugin: 5.48.1

that the own-props-must-be-private and props-muse-be-public tests were failing.

The changes I made to fix them are as follows and it seems from a first glance and resolving some of the aforementioned issues.

To view the changes I made : https://github.com/maldago/stencil-eslint

Let me know if I can do anything to help out.

Hello,

I was playing around with this and discovered that if I updated as follows:
@types/eslint: "8.4.10!
eslint: "8.31.0"
@typescript-eslint/parser: 5.48.1
@typescript-eslint/eslint-plugin: 5.48.1

that the own-props-must-be-private and props-muse-be-public tests were failing.

The changes I made to fix them are as follows and it seems from a first glance and resolving some of the aforementioned issues.

To view the changes I made : https://github.com/maldago/stencil-eslint

Let me know if I can do anything to help out.

Great @maldago! I will have a look next week and see if we can use then. Thanks 🙏

@maldago Big thanks for you comment. I checked your change and created a PR (#84) based on that.
I also tested it in a private project and everything seems to work as expected.

I will close this PR here since we can use the new one as way to move forward.

Thank you all for contributing. 🎉