technomancy / leiningen

Moved to Codeberg; this is a convenience mirror
https://codeberg.org/leiningen/leiningen
Other
7.29k stars 1.61k forks source link

Fix global :exclusions in profile not applying to root :dependencies #2813

Closed jpmonettas closed 10 months ago

jpmonettas commented 10 months ago

This fixes https://github.com/technomancy/leiningen/issues/2811

I ended adding a remove-self-excluded-deps instead of modifying add-global-exclusions to add-and-apply-global-exclusions because I think add-and-apply-global-exclusions conveys that it will somehow apply the exclusions to the deps tree there which is misleading.

jpmonettas commented 10 months ago

The failures and errors shown in CI seems to be the same showing on the previous commit so I don't think they are related to this PR.

technomancy commented 10 months ago

Nice work; this should do the trick. Thank you!