Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#959: Ensure multi-line string are enclosed in blockquotes #961

Merged
merged 1 commit into from
Sep 6, 2024

Conversation

mc1arke
Copy link
Owner

@mc1arke mc1arke commented Sep 3, 2024

The Github Cloud API now returns an error about the input message being malformed, seemingly due to a change in how new lines in messages are being handled. As the use of blockquotes around multi-line messages appears to continue to work, the analysis messages are being wrapped in blockquotes rather than double-quotes where they contain newline characters. This requires an interim measure of cloning the InputObject class from the nodes library to alter the String comparison and wrapping since the library is no longer maintained.

The Github Cloud API now returns an error about the input message being
malformed, seemingly due to a change in how new lines in messages are
being handled. As the use of blockquotes around multi-line messages
appears to continue to work, the analysis messages are being wrapped in
blockquotes rather than double-quotes where they contain newline
characters. This requires an interim measure of cloning the InputObject
class from the nodes library to alter the String comparison and wrapping
since the library is no longer maintained.
Copy link

sonarcloud bot commented Sep 3, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
57.5% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

@jrx-sjg
Copy link

jrx-sjg commented Sep 4, 2024

any chance of get this working soon?

@pedrocarrico
Copy link

@mc1arke I can confirm that this fixes the issue, thank you very much for your fast response.

@zisuu
Copy link

zisuu commented Sep 4, 2024

I can confirm that the snapshot version works here and solved the pull request decoration issue

@qalinn
Copy link

qalinn commented Sep 4, 2024

@mc1arke Any idea when will be merged this PR? Just an estimation if you can provide. Thank you!

@htian-tg
Copy link

htian-tg commented Sep 6, 2024

@mc1arke Really appreciate if this PR can be merged and the new release 1.22.0 or hotfix 1.21.x can be released soon. Thanks.

@mc1arke mc1arke merged commit 6d32805 into master Sep 6, 2024
11 of 12 checks passed
@mc1arke mc1arke deleted the issue/959 branch September 6, 2024 17:59
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.

6 participants