Nick Mancuso

Results 378 comments of Nick Mancuso

> > @Vyom-Yadav I have updated branch filters, let’s try again. > > It's still not working, see #11778 and other latest draft PRs. Ok, updated again, I feel confident...

> please share how filter shuold looks like. I thought I did this yesterday. Need to know where was mistake. ![Screenshot from 2022-06-27 07-30-43](https://user-images.githubusercontent.com/31252532/175931748-bdb91bee-bbcd-4670-bf7d-82108823575a.png)

Ok, we have updated online config to filter by commit message: ![image](https://user-images.githubusercontent.com/31252532/177005393-06f8ccc0-dce4-4144-a5de-e74a8fb9bd93.png) @Vyom-Yadav So we should use `minor: skip TC` as commit message to skip. @romani I am good to...

proof of export of configs: commit from contribution: https://github.com/checkstyle/contribution/commit/7da08e8d90a1e20bc50a9c1a05c09b2457e584fa commit from sonar-checkstyle: https://github.com/checkstyle/sonar-checkstyle/commit/4eaab9ceebcc1990cb528336d209607a4b92adbc commit from main repo: https://github.com/checkstyle/checkstyle/commit/35208d3cf7404145647e07fc05361cbab5546948

I propose that we close this issue after we do the following: 1. Do we want to investigate consolidation of master and PR configs (possible limitation mentioned at https://github.com/checkstyle/checkstyle/issues/11876#issuecomment-1179730313)? 2....

> I think this model proved to be very helpful, and we didn't not finish killing all @Vyom-Yadav This model never worked at all, did it?

TLDR; Moving TeamCity to reliable project based config is too painful at this time, return on investment is just not worth the effort at this point. Expectations for a project...

Clean up from this issue remains. TODO: - [x] Switch TC back to web ui config only - [x] remove all `.teamcity` directories from https://github.com/checkstyle/checkstyle, https://github.com/checkstyle/contribution, and https://github.com/checkstyle/sonar-checkstyle

@romani thanks for reminder, all cleanup is completed.