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

Output (visible) message in case of error (take 2) #32

Merged
merged 1 commit into from
Apr 19, 2022
Merged

Output (visible) message in case of error (take 2) #32

merged 1 commit into from
Apr 19, 2022

Conversation

paulo-ferraz-oliveira
Copy link
Collaborator

I believe this improves on top of #31, which I found to be too verbose after using it from main.

I've (also, now) tested this with a mix of "dependencies with many tags" and "error for unreachable dependency" and the output is as follows.

➜  test ✗ rebar3 update-deps
===> Analyzing applications...
===> Compiling rebar3_depup
===> Looking for dependencies to update in rebar.config...
===> cowboy can be updated from 2.8.0 to 2.9.0
===> ssh: connect to host <host> port <port>: Operation timed out
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
===> Skipping <dep>
===> After applying the changes listed above, don't forget to run rebar3 upgrade.
➜  test ✗

@paulo-ferraz-oliveira paulo-ferraz-oliveira changed the title Feature/more info in case of error Output (visible) message in case of error (take 2) Apr 19, 2022
@elbrujohalcon elbrujohalcon merged commit 5ff9c80 into AdRoll:main Apr 19, 2022
@paulo-ferraz-oliveira paulo-ferraz-oliveira deleted the feature/more-info-in-case-of-error branch April 19, 2022 15:40
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.

2 participants