Nick Mancuso

Results 391 comments of Nick Mancuso

> I don't see a reason of this issue. > > @nrmancuso , do you see a reason? If not , please close issue. I can see how this issue...

> we need to run all samples in our CI on each PR. Can we compile a list of samples?

We need to keep this issue open until we actually update the image version in this repo and restore yaml schema validation: https://github.com/MANISH-K-07/checkstyle/blob/90dd5f0957b24cc929cafdbab1b92c40bc00f245/config/intellij-idea-inspections.xml#L4109

@MANISH-K-07 your new image is live: https://hub.docker.com/layers/checkstyle/idea-docker/jdk11-idea2023.3.4/images/sha256-eefc93eaebcc4d8e747115e539512220d6b41d437f93f2496eda27612b257f18?context=explore ``` ➜ idea-image git:(master) ✗ git log -1 --oneline 80a6139 (HEAD -> master, upstream/master) Issue #14448: Updated IDEA version to v2023.3.4 ➜ idea-image...

> Great !! > > Please let me know when I can update the image in the draft PR #14604 Let’s do it, we will also need to suppress new...

``` ➜ idea-image git:(master) ✗ git log -1 --oneline c22be5f (HEAD -> master, upstream/master) Issue #14448: Migrate IDEA to highest true-scopes release v2022.3.3 ➜ idea-image git:(master) ✗ docker build -t...

@Zopsss please see https://github.com/checkstyle/checkstyle/issues/14273#issuecomment-192281835 We need resolution and approval at https://github.com/checkstyle/checkstyle/issues/14415 before proceeding here.

> Short answer - because current example didn't compile. Yes, this is expected in most cases. > Long - because it is not possible to have static fields in non-public...

We missed removal of exclusions at https://github.com/Lmh-java/checkstyle/blob/16259a82bc779d4adbf329721cfa0eef33d79cd4/pom.xml#L1868 Edit: nevermind, contributor needed to rebase