Change 'ignore'/'ignored' to 'exclude'/'excluded' for consistent naming in public API: CollectIgnoredPathsEvent vs getExcludedPaths()

Discovered in 📌 GitExcluder should use ComposerInspector instead of ComposerUtility Fixed .

I think we should standardize on "ignore", to match the name of the event.

That says, it turns out it's a big lift to rename everything to "ignore". We might wanna standardize on "exclude" just for ease of conversion.

What terminology does Composer Stager use? Should we match that? Why not?

Hmmm...that's a good point.

Composer Stager says "exclusions" and variations thereof and I guess it might make sense to match that, to make it clear and explicit how we integrate with Composer Stager.

So, as per #6 📌 Change 'ignore'/'ignored' to 'exclude'/'excluded' for consistent naming in public API: CollectIgnoredPathsEvent vs getExcludedPaths() Fixed should we change it to 'exclude' everywhere?

@kunal.sachdev yeah lets go with "excluded". thanks

Most of the remaining occurrences of the term "ignore" still refers to this kind of ignoring that #8 decided to consistently name "excluding". Let's make those consistent too.

Also, the issue title needs to be updated since a decision was made.

Still change record. Beyond that, this is ready! 👍

Looks like this needs a rebase/merge conflict fix, but once that's done I'm gonna commit it.

Great work here!

Published https://www.drupal.org/node/3354597 → .

See y'all in 📌 Decide which classes should be internal and/or final — delete ExcludedPathsTrait, make CollectPathsToExcludeEvent richer Fixed !

PB was using class_exists(CollectIgnoredPathsEvent) to check if PM was truly available. I think that may have been a 2.4 vs 2.5 concern. I opened 🐛 Adapt to rename of CollectIgnoredPathsEvent Fixed to swap the event, but if someone from AU could chime in to see if we can simplify that condition, that'd be much appreciated

@phenaproxima already responded: #3356614-2: Adapt to rename of CollectIgnoredPathsEvent → .

Automatically closed - issue fixed for 2 weeks with no activity.