Skip to content

Fix output of alpha channel floats#868

Merged
johnmay merged 3 commits intocdk:masterfrom
egonw:fix/bug860
Jun 7, 2022
Merged

Fix output of alpha channel floats#868
johnmay merged 3 commits intocdk:masterfrom
egonw:fix/bug860

Conversation

@egonw
Copy link
Copy Markdown
Member

@egonw egonw commented Jun 6, 2022

Fixed #860

@egonw egonw requested a review from johnmay June 6, 2022 11:24
Comment thread app/depict/src/main/java/org/openscience/cdk/depict/SvgDrawVisitor.java Outdated
@johnmay
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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