Skip to content

Conversation

egonw
Copy link
Member

@egonw egonw commented Jun 6, 2022

Fixed #860

@egonw egonw requested a review from johnmay June 6, 2022 11:24
@johnmay
Copy link
Member

johnmay commented Jun 7, 2022

Shall we just put the sonar cloud key in the GH action?

@johnmay johnmay merged commit b12da73 into cdk:master Jun 7, 2022
@egonw
Copy link
Member Author

egonw commented Jun 7, 2022

Shall we just put the sonar cloud key in the GH action?

Only as a hidden key in the GitHub organization and then use ${XXX} in the action.

@johnmay
Copy link
Member

johnmay commented Jun 7, 2022

Well that's what we have, the issue is even you can't get it to run with a PR from a branch in your fork

@johnmay
Copy link
Member

johnmay commented Jun 7, 2022

I think there is a way to do it with GITHUB_TOKEN instead of SONAR_TOKEN... where by user details allow access control

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.

[SvgDrawVisitor] - Problem with Locale when String.format(...) is used
2 participants