Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add --exclude-dependents boolean flag #17

Closed
wants to merge 2 commits into from
Closed

Conversation

br0p0p
Copy link

@br0p0p br0p0p commented Sep 24, 2021

Hi,

Thanks for creating the changed plugin!

We're using this in CI at work and we don't version everything in lock step so I'd like a way to avoid including dependencies in the list of changed workspaces.

The default behavior is unchanged but with this update I can do the following:

yarn changed list --git-range origin/master --exclude-dependents

Definitely open to naming changes on this. Let me know what you think.


I think the root issue might actually be that in our monorepo all packages rely on the root package so any changes there count as a "dependent" change for all packages. Maybe adding a flag to ignore the root package's changes when calculating dependents is the better way to go. I see someone already brought that up in #6

Interested to hear your thoughts.

@br0p0p br0p0p changed the title Add --include-dependents boolean flag Add --exclude-dependents boolean flag Sep 24, 2021
@br0p0p br0p0p closed this Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant