exercism / website

The codebase for Exercism's website.
https://exercism.org
GNU Affero General Public License v3.0
408 stars 120 forks source link

ArgumentError in spi/tooling_jobs#update #1156

Open iHiD opened 3 years ago

iHiD commented 3 years ago

Error in v3

ArgumentError in spi/tooling_jobs#update malformed format string - %C

View on Bugsnag

Stacktrace

app/models/submission/analysis.rb:52 - %
app/models/submission/analysis.rb:52 - block in comments
app/models/submission/analysis.rb:43 - map
app/models/submission/analysis.rb:43 - comments
app/models/submission.rb:119 - analyzer_feedback
app/models/iteration.rb:11 - analyzer_feedback
app/serializers/serialize_iteration.rb:24 - call
app/channels/solution_channel.rb:13 - block in broadcast!
app/channels/solution_channel.rb:13 - map
app/channels/solution_channel.rb:13 - broadcast!
app/models/solution.rb:181 - broadcast!
app/models/iteration.rb:55 - broadcast!
app/commands/submission/representation/process.rb:32 - call
app/commands/tooling_job/process.rb:18 - process_representer_job!
app/commands/tooling_job/process.rb:8 - call
app/controllers/spi/tooling_jobs_controller.rb:4 - update

View full stacktrace

Created automatically via Bugsnag

iHiD commented 3 years ago

@exercism/elixir This is an issue with the analyzer:

"{\"comments\":[{\"comment\":\"elixir.pacman-rules.use_strictly_boolean_operators\",\"type\":\"essential\"}],\"summary\":\"Check the comments for things to fix. 🛠\"}"},

I'm guessing there's some parse error in the file? Maybe something not escaped?

iHiD commented 3 years ago

Yeah - the % needs to be escape in the URL of # of https://raw.githubusercontent.com/exercism/website-copy/main/analyzer-comments/elixir/pacman-rules/use_strictly_boolean_operators.md

iHiD commented 2 years ago

An error linked to this issue has been reopened in Bugsnag Octokit::UnprocessableEntity in spi/tooling_jobs#update

neenjaw commented 2 years ago

Do you know which exercise it was analyzing

angelikatyborska commented 2 years ago

My bet would be on this rust comment from @ErikSchierboom that got merged at the same time as this issue got reopened: https://github.com/exercism/website-copy/pull/2128/files#diff-e0c4882cc1dd17448fa1fd4de20cb7649d3347918b0f272b8272791e9bae955fR3

ErikSchierboom commented 2 years ago

@iHiD Should it be escaped as %%?

iHiD commented 2 years ago

Yes. I believe so.

ErikSchierboom commented 2 years ago

PR submitted