Skip to content

Update FreeHepWrapper.java - #851

Merged
johnmay merged 1 commit into
cdk:masterfrom
OBrink:patch-1
Apr 7, 2022
Merged

Update FreeHepWrapper.java#851
johnmay merged 1 commit into
cdk:masterfrom
OBrink:patch-1

Conversation

@OBrink

@OBrink OBrink commented Apr 6, 2022

Copy link
Copy Markdown
Contributor

Typo correction

@egonw
egonw self-requested a review April 6, 2022 12:14
@egonw

egonw commented Apr 6, 2022

Copy link
Copy Markdown
Member

@johnmay, this one is a simple typo fix. Sonarcloud fails, because it doesn't work on remote branches,

@johnmay
johnmay merged commit 67daab2 into cdk:master Apr 7, 2022
@johnmay

johnmay commented Apr 7, 2022

Copy link
Copy Markdown
Member

I am curious of the solution there - should we just put the key in bare in the YAML config?

@egonw

egonw commented Apr 7, 2022

Copy link
Copy Markdown
Member

Not sure... we should be able to google this. I'll do some searching this week.

@johnmay

johnmay commented Apr 7, 2022

Copy link
Copy Markdown
Member

It does seem to be setup correctly - or how SonarCloud document.

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.

3 participants