Skip to content

Commit 6cb4a13

Browse files
Maintenance: fix some code warnings (#214)
Co-authored-by: Chris Kilding <[email protected]>
1 parent 8f63a52 commit 6cb4a13

File tree

7 files changed

+13
-24
lines changed

7 files changed

+13
-24
lines changed

.gitignore

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,11 @@
2121
# virtual machine crash logs, see http://www.java.com/en/download/help/error_hotspot.xml
2222
hs_err_pid*
2323

24+
# Eclipse IDE files
25+
.classpath
26+
.project
27+
.settings
28+
2429
# Others
2530

2631
.DS_Store

src/main/java/io/jenkins/plugins/credentials/secretsmanager/AwsCredentialsProvider.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ public <C extends Credentials> List<C> getCredentials(@Nonnull Class<C> type,
6161

6262
@Override
6363
public CredentialsStore getStore(ModelObject object) {
64-
return object == Jenkins.getInstance() ? store : null;
64+
return object == Jenkins.get() ? store : null;
6565
}
6666

6767
@Override

src/main/java/io/jenkins/plugins/credentials/secretsmanager/AwsCredentialsStore.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,23 +37,23 @@ public AwsCredentialsStore(AwsCredentialsProvider provider) {
3737
@Nonnull
3838
@Override
3939
public ModelObject getContext() {
40-
return Jenkins.getInstance();
40+
return Jenkins.get();
4141
}
4242

4343
@Override
4444
public boolean hasPermission(@NonNull Authentication authentication,
4545
@NonNull Permission permission) {
4646
return CredentialsProvider.VIEW.equals(permission)
47-
&& Jenkins.getInstance().getACL().hasPermission(authentication, permission);
47+
&& Jenkins.get().getACL().hasPermission(authentication, permission);
4848
}
4949

5050
@Nonnull
5151
@Override
5252
public List<Credentials> getCredentials(@NonNull Domain domain) {
5353
// Only the global domain is supported
5454
if (Domain.global().equals(domain)
55-
&& Jenkins.getInstance().hasPermission(CredentialsProvider.VIEW)) {
56-
return provider.getCredentials(Credentials.class, Jenkins.getInstance(), ACL.SYSTEM);
55+
&& Jenkins.get().hasPermission(CredentialsProvider.VIEW)) {
56+
return provider.getCredentials(Credentials.class, Jenkins.get(), ACL.SYSTEM);
5757
} else {
5858
return Collections.emptyList();
5959
}

src/main/java/io/jenkins/plugins/credentials/secretsmanager/config/credentialsProvider/CredentialsProvider.java

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -16,16 +16,11 @@ public abstract class CredentialsProvider extends AbstractDescribableImpl<Creden
1616
public abstract AWSCredentialsProvider build();
1717

1818
public abstract static class DescriptorImpl extends Descriptor<CredentialsProvider> {
19-
/**
20-
* {@inheritDoc}
21-
*/
19+
2220
protected DescriptorImpl() {
2321
super();
2422
}
2523

26-
/**
27-
* {@inheritDoc}
28-
*/
2924
protected DescriptorImpl(Class<? extends CredentialsProvider> clazz) {
3025
super(clazz);
3126
}

src/main/java/io/jenkins/plugins/credentials/secretsmanager/config/transformer/description/DescriptionTransformer.java

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,16 +18,11 @@ public abstract class DescriptionTransformer extends AbstractDescribableImpl<Des
1818
public abstract String transform(String str);
1919

2020
public abstract static class DescriptorImpl extends Descriptor<DescriptionTransformer> {
21-
/**
22-
* {@inheritDoc}
23-
*/
21+
2422
protected DescriptorImpl() {
2523
super();
2624
}
2725

28-
/**
29-
* {@inheritDoc}
30-
*/
3126
protected DescriptorImpl(Class<? extends DescriptionTransformer> clazz) {
3227
super(clazz);
3328
}

src/main/java/io/jenkins/plugins/credentials/secretsmanager/config/transformer/name/NameTransformer.java

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,16 +18,11 @@ public abstract class NameTransformer extends AbstractDescribableImpl<NameTransf
1818
public abstract String transform(String str);
1919

2020
public abstract static class DescriptorImpl extends Descriptor<NameTransformer> {
21-
/**
22-
* {@inheritDoc}
23-
*/
21+
2422
protected DescriptorImpl() {
2523
super();
2624
}
2725

28-
/**
29-
* {@inheritDoc}
30-
*/
3126
protected DescriptorImpl(Class<? extends NameTransformer> clazz) {
3227
super(clazz);
3328
}

src/test/java/io/jenkins/plugins/credentials/secretsmanager/util/assertions/WorkflowRunAssert.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
package io.jenkins.plugins.credentials.secretsmanager.util.assertions;
22

33
import org.assertj.core.api.AbstractAssert;
4-
import org.jenkinsci.plugins.plaincredentials.FileCredentials;
54
import org.jenkinsci.plugins.workflow.job.WorkflowRun;
65
import hudson.model.Result;
76

0 commit comments

Comments
 (0)