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

improve warning messages, especially with ~branch #2585

Merged
merged 2 commits into from
Feb 15, 2023

Conversation

WebFreak001
Copy link
Member

Before:
before image

     Warning WARNING: A deprecated branch based version specification is used for the dependency gitver. Please use numbered versions instead. Also note that you can still use the dub.selections.json file to override a certain dependency to use a branch instead.

After:
after image

     Warning Dependency 'gitver' depends on git branch '~main', which is deprecated.
             Specify the git repository and commit hash in your dub.json:
             "gitver": {"repository": "git <git url>", "version": "<commit>"}

Also removes duplicate Warning WARNING: ... from two other messages

@Geod24 Geod24 merged commit 6948b09 into dlang:master Feb 15, 2023
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