Nick Mancuso

Results 378 comments of Nick Mancuso

Yeah, the issue description really doesn’t capture the large amount of work that will need to be done to support this.

@romani please show the full document format in valid yaml. Also, we should use yaml/yml file extension

IMO, project properties should not be array elements.

yaml structure should be: ```yaml projects: - checkstyle: scm: git url: https://github.com/checkstyle/checkstyle.git branch: master enabled: true excludes: - **/.ci-temp/**/* - **/resources-noncompilable/**/asttreestringprinter/**/* - **/resources-noncompilable/**/filefilters/**/* ... ``` Override can be provided by:...

> @nrmancuso , I probably see right now point of confusion. I updated description to show that config will be list of projects, not a single project. I am not...

> in this case I do not undersatnd what `--projects.checkstyle.enabled=false` is going to do. It would override the value of this property.

> and we need `--projects.sevntu-checkstyle.enable=false` ??? Am I reading you mind properly ? Sure. We shouldn’t have to have more than one project file anywhere. Default can be disabled, and...

Ok, we can use circular dependency on project list in arguments, but in either case, we shouldn’t have a bunch of these files in the config repo; this sort of...