Skip to content

Update FreeHepWrapper.java#851

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

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

Conversation

@OBrink
Copy link
Copy Markdown
Contributor

@OBrink OBrink commented Apr 6, 2022

Typo correction

@egonw egonw self-requested a review April 6, 2022 12:14
@egonw
Copy link
Copy Markdown
Member

egonw commented Apr 6, 2022

@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
Copy link
Copy Markdown
Member

johnmay commented Apr 7, 2022

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

@egonw
Copy link
Copy Markdown
Member

egonw commented Apr 7, 2022

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

@johnmay
Copy link
Copy Markdown
Member

johnmay commented Apr 7, 2022

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