--exclude flag could use better documentation

[marrip]

Hey,

I was a bit confused on how the --exclude flag works. I tried different approaches like defining it multiple times or separating the different regexes with spaces but nothing worked and the docs did not make any suggestion on how it is handled. I checked the code and realized that it should be --exclude "regex1|regex2|regex3". I think it would be beneficial for users to have this somewhere explained to ease usage. Thank you 🙂

[generalmimon]

Yeah, currently only the last -exclude setting gets picked up and any previous setting is silently ignored (NB: no \.rb$ in "Exclude Regexp"):

$ ec -verbose -exclude '\.rb$' -exclude '\.gemspec$'
Exclude Regexp: \.gemspec$|^\.yarn/|^yarn\.lock$|^package-lock\.json$|^composer\.lock$|^Cargo\.lock$|^Gemfile\.lock$|^\.pnp\.cjs$|^\.pnp\.js$|^\.pnp\.loader\.mjs$|\.snap$|\.otf$|\.woff$|\.woff2$|\.eot$|\.ttf$|\.gif$|\.png$|\.jpg$|\.jpeg$|\.webp$|\.avif$|\.mp4$|\.wmv$|\.svg$|\.ico$|\.bak$|\.bin$|\.pdf$|\.zip$|\.gz$|\.tar$|\.7z$|\.bz2$|\.log$|\.patch$|\.css\.map$|\.js\.map$|min\.css$|min\.js$

Ironically, this project's own Makefile doesn't account for this either:

editorconfig-checker/Makefile

Lines 52 to 56 in 585ef52

	run: build ## Build and run bin/ec
	@./bin/ec --exclude "\\.git" --exclude "\\.exe$$"
	run-verbose: build ## Build and run bin/ec --verbose
	@./bin/ec --verbose --exclude "\\.git" --exclude "\\.exe$$"

It would be great if multiple -excludes were all picked up and merged into the regexp as alternatives (like the Exclude array in JSON configuration), as many of us would intuitively expect. An inferior alternative (but still better than the current silent "last wins" overwrite) would be to detect multiple -exclude settings and inform the user that only the last one was applied - but I guess that the proper fix of just adding support for multiple -excludes would actually be easier to implement than this ;)

[mstruebing]

Anyone willing to create a PR for this?