fix: enable the reporting of parsing related problems when running deno lint

[HasanAlrimawi]

Supported reporting of syntax/parsing errors.

The commit is made to address issue #23437

Code changes

Added check for ParsedSource.diagnostics() and when it's not empty, all parse diagnostic errors will be logged/displayed.

PS: The change was within fn handle_lint_result() to log any found parse or lint diagnostic errors, since this function is called after any lint is done.

[bartlomieju]

Thanks, could you add a test to tests/specs/lint/ that exercises this change?

[HasanAlrimawi]

Hello @bartlomieju ,
I added a test to exercise the change, but the PR check (ci / lint debug linux-x86_64) failed because the test I added includes syntax error (unterminated string constant) in the script file. However, this syntax error was intentionally added to show that the change done can detect this syntax error.
Is there a way to overcome this from my or your side, or should I just remove the test?

[HasanAlrimawi]

Hello @bartlomieju ,
Just as a notification.
What do you think is best to do with the case here

[bartlomieju]

@HasanAlrimawi you need to update .dprint.json file and add the file you added to the "excludes" config so that it's not being formatted.

[dsherret]

LGTM. Thanks!