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

Stop hiding credentials configuration category #1752

Merged
merged 1 commit into from Oct 1, 2021

Conversation

timja
Copy link
Member

@timja timja commented Oct 1, 2021

It was returning empty string because the symbol guessing logic was
stripping everything

A symbol was added in jenkinsci/credentials-plugin#144 back in April 2020

Part of fixing jenkinsci/azure-keyvault-plugin#92 (other part in credentials plugin)

It was returning empty string because the symbol guessing logic was
stripping everything

A symbol was added in jenkinsci/credentials-plugin#144 back in April 2020
@codecov
Copy link

codecov bot commented Oct 1, 2021

Codecov Report

Merging #1752 (5fb5ad5) into master (7b958de) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1752      +/-   ##
============================================
- Coverage     80.81%   80.80%   -0.02%     
+ Complexity      836      835       -1     
============================================
  Files            71       71              
  Lines          2466     2464       -2     
  Branches        346      345       -1     
============================================
- Hits           1993     1991       -2     
  Misses          363      363              
  Partials        110      110              
Impacted Files Coverage Δ
...ators/GlobalConfigurationCategoryConfigurator.java 87.80% <100.00%> (-0.57%) ⬇️

@timja timja merged commit 0246dc8 into jenkinsci:master Oct 1, 2021
@timja timja deleted the stop-hiding-credentials-category branch October 1, 2021 09:24
Copy link
Member

@jetersen jetersen left a comment

Choose a reason for hiding this comment

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

Belated LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A PR that adds a feature - used by Release Drafter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants