Skip to content
New issue

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

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

Already on GitHub? # to your account

keys: add support for --ignore-src #686

Merged

Conversation

kyrofa
Copy link
Contributor

@kyrofa kyrofa commented Jun 6, 2019

Currently --ignore-src is only supported by the check and install verbs. This PR resolves #685 by adding support in the keys verb as well, displaying dependencies that are not already satisfied by other packages in the workspace.

Currently `--ignore-src` is only supported by the `check` and `install`
verbs. Add support in the `keys` verb as well, displaying
dependencies that are not already satisfied by other packages in the
workspace.

Resolve ros-infrastructure#685

Signed-off-by: Kyle Fazzari <kyrofa@ubuntu.com>
@kyrofa kyrofa force-pushed the feature/685/keys_ignore_src branch from 0239440 to e9b3a56 Compare June 6, 2019 21:03
@codecov-io
Copy link

codecov-io commented Jun 6, 2019

Codecov Report

Merging #686 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #686      +/-   ##
==========================================
+ Coverage   75.46%   75.47%   +<.01%     
==========================================
  Files          32       32              
  Lines        2967     2968       +1     
==========================================
+ Hits         2239     2240       +1     
  Misses        728      728
Impacted Files Coverage Δ
src/rosdep2/main.py 48.58% <100%> (+0.09%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a7112a...e9b3a56. Read the comment docs.

@kyrofa
Copy link
Contributor Author

kyrofa commented Jun 10, 2019

@nuclearsandwich curious to get your thoughts on this.

@cottsay cottsay merged commit ab1f06e into ros-infrastructure:master Jun 12, 2019
@kyrofa kyrofa deleted the feature/685/keys_ignore_src branch June 12, 2019 16:02
@kyrofa
Copy link
Contributor Author

kyrofa commented Jul 18, 2019

Hey @tfoote, any chance we can get this released? We're carrying some patches that we can drop once that happens.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Apply --ignore-src to keys as well
4 participants