Skip to content

fix: Define a constant #62

Merged
LinuxSuRen merged 1 commit into
LinuxSuRen:masterfrom
wongearl:fix/constant
May 8, 2023
Merged

fix: Define a constant #62
LinuxSuRen merged 1 commit into
LinuxSuRen:masterfrom
wongearl:fix/constant

Conversation

@wongearl

@wongearl wongearl commented May 8, 2023

Copy link
Copy Markdown
Contributor

Define a constant instead of duplicating this literal "Bearer {{.login.data.access_token}}" 3 times.

@sonarqubecloud

sonarqubecloud Bot commented May 8, 2023

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@wongearl

wongearl commented May 8, 2023

Copy link
Copy Markdown
Contributor Author

The test report skipped like it did not take effect.

@LinuxSuRen
LinuxSuRen merged commit bb74504 into LinuxSuRen:master May 8, 2023
@wongearl
wongearl deleted the fix/constant branch May 9, 2023 10:47
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