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

Replace coursier-resolve goal with generate-lockfiles #14211

Merged

Conversation

Eric-Arellano
Copy link
Contributor

[ci skip-rust]

# Rust tests and lints will be skipped. Delete if not intended.
[ci skip-rust]

# Building wheels and fs_util will be skipped. Delete if not intended.
[ci skip-build-wheels]
@Eric-Arellano Eric-Arellano force-pushed the generate-lockfiles-jvm-user branch from 51b4354 to c6e3aa5 Compare January 19, 2022 21:19
@@ -0,0 +1,142 @@
# Copyright 2021 Pants project contributors (see CONTRIBUTORS.md).
# Licensed under the Apache License, Version 2.0 (see LICENSE).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

These tests are very similar to coursier_integration_test.py, but they're more direct. We don't run a goal_rule anymore, and they instead test the rules directly.

# Rust tests and lints will be skipped. Delete if not intended.
[ci skip-rust]

# Building wheels and fs_util will be skipped. Delete if not intended.
[ci skip-build-wheels]
Copy link
Member

@stuhood stuhood left a comment

Choose a reason for hiding this comment

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

Huzzah! Thanks a lot @Eric-Arellano.

src/python/pants/jvm/goals/lockfile.py Outdated Show resolved Hide resolved
Eric-Arellano and others added 2 commits January 19, 2022 14:59
…files-jvm-user

# Conflicts:
#	src/python/pants/backend/java/compile/javac_test.py
#	src/python/pants/jvm/goals/coursier.py
#	src/python/pants/jvm/goals/coursier_integration_test.py
#	src/python/pants/jvm/goals/lockfile.py

# Rust tests and lints will be skipped. Delete if not intended.
[ci skip-rust]

# Building wheels and fs_util will be skipped. Delete if not intended.
[ci skip-build-wheels]
@Eric-Arellano Eric-Arellano enabled auto-merge (squash) January 19, 2022 23:51
# Rust tests and lints will be skipped. Delete if not intended.
[ci skip-rust]

# Building wheels and fs_util will be skipped. Delete if not intended.
[ci skip-build-wheels]
@Eric-Arellano Eric-Arellano merged commit 1080d7b into pantsbuild:main Jan 20, 2022
@Eric-Arellano Eric-Arellano deleted the generate-lockfiles-jvm-user branch January 20, 2022 01:37
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