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.

Sign up for GitHub

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

Jump to bottom

Remove duplicate dependency of awaitility. #10979

Merged
merged 1 commit into from Jun 19, 2021
Merged

Remove duplicate dependency of awaitility. #10979

merged 1 commit into from Jun 19, 2021

Conversation

yangl
Copy link
Contributor

@yangl yangl commented Jun 19, 2021

Motivation

[INFO] Scanning for projects...
[WARNING] 
[WARNING] Some problems were encountered while building the effective model for org.apache.pulsar:pulsar-metadata:jar:2.9.0-SNAPSHOT
[WARNING] 'dependencies.dependency.(groupId:artifactId:type:classifier)' must be unique: org.awaitility:awaitility:jar -> duplicate declaration of version (?) @ line 76, column 17
[WARNING] 
[WARNING] It is highly recommended to fix these problems because they threaten the stability of your build.
[WARNING] 
[WARNING] For this reason, future Maven versions might no longer support building such malformed projects.
[WARNING]

The warn was introduced from this change 21ea76c

Modifications

delete the duplicate awaitility dependency.

@yangl
Remove duplicate dependency of awaitility.
239528e 
Signed-off-by: YANGLiiN <ielin@qq.com>
@lhotari lhotari merged commit fec6c58 into apache:master Jun 19, 2021
@yangl yangl deleted the mvn_warn branch June 19, 2021 13:00
yangl added a commit to yangl/pulsar that referenced this pull request Jun 23, 2021
@yangl
Remove duplicate dependency of awaitility. (apache#10979)
c52e336 
Signed-off-by: YANGLiiN <ielin@qq.com>
bharanic-dev pushed a commit to bharanic-dev/pulsar that referenced this pull request Mar 18, 2022
@yangl @ciaocloud
Remove duplicate dependency of awaitility. (apache#10979)
c70d6ad 
Signed-off-by: YANGLiiN <ielin@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lhotari lhotari lhotari approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@yangl @lhotari