-
Notifications
You must be signed in to change notification settings - Fork 240
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
Fix integration tests #2730
Fix integration tests #2730
Conversation
130662f
to
d259d2f
Compare
d259d2f
to
af06627
Compare
af06627
to
a3aaa4d
Compare
a3aaa4d
to
e8125d5
Compare
d0ed32a
to
4a4badb
Compare
4a4badb
to
9172339
Compare
9172339
to
9e6a9c8
Compare
9e6a9c8
to
5623854
Compare
5623854
to
9235dde
Compare
9235dde
to
016a003
Compare
016a003
to
94b2098
Compare
94b2098
to
78d88cc
Compare
78d88cc
to
0f797a2
Compare
0f797a2
to
8d6d68d
Compare
8d6d68d
to
1ca7beb
Compare
1ca7beb
to
f9f11ad
Compare
f9f11ad
to
d6d9467
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume that the removal of the archive indexing also means that the below flag no longer works from a specific version of Artifactory.
If this is the case, let's also add the following prefix to the flag's description -
This option stopped being supported since version XXX for Artifactory.
(replace XXX with the right version).
Let's also update the online documentation with that description.
archiveEntries: cli.StringFlag{
Name: archiveEntries,
Usage: "[Optional] If specified, only archive artifacts containing entries matching this pattern are matched. You can use wildcards to specify multiple artifacts.` `",
},
dev
branch.go vet ./...
.go fmt ./...
.Update the tests to align with changes introduced in Artifactory 7.90+:
http://127.0.0.1:8081/access
are no longer functional. Instead, usehttp://127.0.0.1:8082/access
.Additionally, in this pull request, the access tests are now executed locally on 127.0.0.1 instead of the SaaS server.