multimod: ignore excluded-modules when -a flag is enabled

[codeboten]

This allows users of the sync command to sync all modules in a monorepo, including those listed in the excluded-modules. This is useful for repositories where some modules may not yet be ready for releasing (therefore listed under excluded-modules) but their dependencies still need to be managed via multimod.

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (49952c1) 63.51% compared to head (63a96fe) 63.65%.

Files	Patch %	Lines
multimod/internal/sync/sync.go	66.66%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #442      +/-   ##
==========================================
+ Coverage   63.51%   63.65%   +0.14%     
==========================================
  Files          39       39              
  Lines        2250     2259       +9     
==========================================
+ Hits         1429     1438       +9     
  Misses        677      677              
  Partials      144      144              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pellared]

Do we need this flag? Is there any reason why it shouldn't we simply change the behavior without introducing the flag?

[codeboten]

Do we need this flag? Is there any reason why it shouldn't we simply change the behavior without introducing the flag?

It's a good question, I didn't want to assume that all users would benefit from this change, but thinking back on it, maybe it is desirable for all users to update sync deps even for excluded modules. I could go either ways with this.

[codeboten]

@pellared after re-thinking this, I changed the behaviour for anyone that uses the -a flag