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

Feature Request: Allow sync (or bumps) for specific dependencies #11

Closed
ciriarte opened this issue Aug 11, 2020 · 5 comments
Closed

Feature Request: Allow sync (or bumps) for specific dependencies #11

ciriarte opened this issue Aug 11, 2020 · 5 comments
Labels
enhancement This issue is a feature request in-progress Work has begun by a community member or a maintainer; this issue may be included in a future release

Comments

@ciriarte
Copy link

Context

vendir currently supports the --directory flag:

$ vendir sync -d my_dependency

But it doesn't save the lock file. I would like vendir to be able to isolate sync/bumps to a specific dependency.

@ciriarte ciriarte changed the title Allow sync (or bumps) for specific dependencies Feature Request: Allow sync (or bumps) for specific dependencies Aug 11, 2020
@cppforlife cppforlife added the enhancement This issue is a feature request label Aug 11, 2020
@cari-lynn
Copy link

Hey @ciriarte!
Let us know if this captures your intent.

Currently vendir sync only syncs an entire vendir.yml file.
You can use vendir sync --directory to override contents of particular directories by pointing them to local directories. However, this does not update the lock file.

To support syncing only a subset of the dependencies in the vendir.yml, vendir sync would provide the ability to target the given dependencies, and update the vendir.lock.yml with the new metadata.

@cppforlife
Copy link
Contributor

@ciriarte we wouldnt expect to modify anything other dependencies -- even if they were modified on disk, intentionally or unintentionally -- right?

@ciriarte
Copy link
Author

@ciriarte we wouldnt expect to modify anything other dependencies -- even if they were modified on disk, intentionally or unintentionally -- right?

My expectation is that vendir will only update the directory it is managing and I've instructed in the vendir sync -d some_dir call.

@ciriarte
Copy link
Author

Currently vendir sync only syncs an entire vendir.yml file.
You can use vendir sync --directory to override contents of particular directories by pointing them to local directories. However, this does not update the lock file.

@pivotal-dean

My understanding is that vendir sync -d has a couple of options per the help command:

  1. It does sync local dirs if used with dir/sub-dir[=local-dir] I'm not really interested in this workflow at the moment 🚫
  2. It synchronizes with a remote if the optional [=local-dir] is omitted. This does something close to my desired outcome

If my understanding of 2 is correct, then you captured my wishes perfectly in the following sentence:

To support syncing only a subset of the dependencies in the vendir.yml, vendir sync would provide the ability to target the given dependencies, and update the vendir.lock.yml with the new metadata.

@cppforlife cppforlife added the in-progress Work has begun by a community member or a maintainer; this issue may be included in a future release label Aug 17, 2020
@cppforlife
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This issue is a feature request in-progress Work has begun by a community member or a maintainer; this issue may be included in a future release
Projects
None yet
Development

No branches or pull requests

3 participants