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.

Sign up for GitHub

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

Jump to bottom

Fix MixMachine.Format.Sarif.normalize/1 for {line, col} #17

Open

florius0 wants to merge 1 commit into hauleth:master from florius0:fix-normalize

florius0 commented Nov 26, 2024 •

edited

Loading

Hi!

As of now mix compile.machine at

Erlang/OTP 26 [erts-14.2.5] [source] [64-bit] [smp:12:12] [ds:12:12:10] [async-threads:1] [dtrace]

Elixir 1.17.2 (compiled with Erlang/OTP 26)

fails due to match error in MixMachine.Format.Sarif.normalize/1.

I've patched it to handle {line, col} tuples.

Tho I'm unsure of SARIF requirements, however judging by the way it is implemented RN, it should be fine


          fix: MixMachine.Format.Sarif.normalize/1 for {line, col}

28b2c14

github-actions bot added the scope/format label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels