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

tuf/repository_tool: Return delegated bin_name during modifications #1040

Conversation

woodruffw
Copy link
Contributor

@woodruffw woodruffw commented Jun 1, 2020

Fixes issue #: None (raised on chat).

Description of the changes being introduced by the pull request:

This makes it easier for consumers of repository_tool to mark the
appropriate delegated bin as dirty when using delegated targets.

Please verify and check that the pull request fulfills the following
requirements
:

  • The code follows the Code Style Guidelines
  • Tests have been added for the bug fix or new feature
  • Docs have been added for the bug fix or new feature

I wasn't sure how to go about adding tests for this; I can look into the test_repository_tool.py tests to see what can be done.

This makes it easier for consumers of repository_tool to mark the
appropriate delegated bin as dirty when using delegated targets.

Signed-off-by: William Woodruff <william@trailofbits.com>
@woodruffw woodruffw force-pushed the ww/return-bin-name-when-delegating branch from 78decd0 to 65fd02c Compare June 1, 2020 17:47
@woodruffw
Copy link
Contributor Author

woodruffw commented Jun 1, 2020

N.B.: This is obviated by #1038, but provides a stopgap until that's ready and merged (or a larger refactor occurs).

Signed-off-by: William Woodruff <william@trailofbits.com>
@woodruffw woodruffw force-pushed the ww/return-bin-name-when-delegating branch from 6043f6a to 4327a98 Compare June 1, 2020 18:01
Signed-off-by: William Woodruff <william@trailofbits.com>
Copy link
Contributor

@mnm678 mnm678 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM as a stopgap until #1038 is ready

@mnm678 mnm678 merged commit a354fc0 into theupdateframework:develop Jun 1, 2020
@woodruffw woodruffw deleted the ww/return-bin-name-when-delegating branch June 1, 2020 23:01
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.

3 participants