style(svelte-check): make human output more concise and readable

[YimingIsCOLD]

This PR makes the human output of svelte-check more concise and readable.

TLDR:

• Remove ==================================== separator from the output
• Remove unnecessary leading newlines at the start of the output
• Trimmed trailing newlines at the end of code blocks when using the --output=human-verbose flag

--output=human

No Errors or Warnings

Before

====================================
svelte-check found 0 errors and 0 warnings

After

svelte-check found 0 errors and 0 warnings

Single Error/Warning

Before

/Users/yiming/workspace/somekit/src/routes/+layout.svelte:10:3
Warn: Unused CSS selector ".container"
https://svelte.dev/e/css_unused_selector (svelte)

====================================
svelte-check found 0 errors and 1 warning in 1 file

After

/Users/yiming/workspace/somekit/src/routes/+layout.svelte:10:3
Warn: Unused CSS selector ".container"
https://svelte.dev/e/css_unused_selector (svelte)

svelte-check found 0 errors and 1 warning in 1 file

Multiple Errors/Warnings

Before

/Users/yiming/workspace/somekit/src/routes/+layout.svelte:10:3
Warn: Unused CSS selector ".container"
https://svelte.dev/e/css_unused_selector (svelte)

/Users/yiming/workspace/somekit/src/routes/+page.svelte:4:3
Warn: Unused CSS selector ".container"
https://svelte.dev/e/css_unused_selector (svelte)

====================================
svelte-check found 0 errors and 2 warnings in 2 files

After

/Users/yiming/workspace/somekit/src/routes/+layout.svelte:10:3
Warn: Unused CSS selector ".container"
https://svelte.dev/e/css_unused_selector (svelte)

/Users/yiming/workspace/somekit/src/routes/+page.svelte:4:3
Warn: Unused CSS selector ".container"
https://svelte.dev/e/css_unused_selector (svelte)

svelte-check found 0 errors and 2 warnings in 2 files

--output=human-verbose

No Errors or Warnings

Before

 
====================================
Loading svelte-check in workspace: /Users/yiming/workspace/somekit
Getting Svelte diagnostics...

====================================
svelte-check found 0 errors and 0 warnings

After

Loading svelte-check in workspace: /Users/yiming/workspace/somekit
Getting Svelte diagnostics...

svelte-check found 0 errors and 0 warnings

Single Error/Warning

Before

 
====================================
Loading svelte-check in workspace: /Users/yiming/workspace/somekit
Getting Svelte diagnostics...

/Users/yiming/workspace/somekit/src/routes/+layout.svelte:10:3
Warn: Unused CSS selector ".container"
https://svelte.dev/e/css_unused_selector (svelte)
<style>
  .container {
    background-color: red;


====================================
svelte-check found 0 errors and 1 warning in 1 file

After

Loading svelte-check in workspace: /Users/yiming/workspace/somekit
Getting Svelte diagnostics...

/Users/yiming/workspace/somekit/src/routes/+layout.svelte:10:3
Warn: Unused CSS selector ".container"
https://svelte.dev/e/css_unused_selector (svelte)
<style>
  .container {
    background-color: red;

svelte-check found 0 errors and 1 warning in 1 file

Multiple Errors/Warnings

Before

 
====================================
Loading svelte-check in workspace: /Users/yiming/workspace/somekit
Getting Svelte diagnostics...

/Users/yiming/workspace/somekit/src/routes/+layout.svelte:10:3
Warn: Unused CSS selector ".container"
https://svelte.dev/e/css_unused_selector (svelte)
<style>
  .container {
    background-color: red;


/Users/yiming/workspace/somekit/src/routes/+page.svelte:4:3
Warn: Unused CSS selector ".container"
https://svelte.dev/e/css_unused_selector (svelte)
<style>
  .container {
    background-color: red;


====================================
svelte-check found 0 errors and 2 warnings in 2 files

After

Loading svelte-check in workspace: /Users/yiming/workspace/somekit
Getting Svelte diagnostics...

/Users/yiming/workspace/somekit/src/routes/+layout.svelte:10:3
Warn: Unused CSS selector ".container"
https://svelte.dev/e/css_unused_selector (svelte)
<style>
  .container {
    background-color: red;

/Users/yiming/workspace/somekit/src/routes/+page.svelte:4:3
Warn: Unused CSS selector ".container"
https://svelte.dev/e/css_unused_selector (svelte)
<style>
  .container {
    background-color: red;

svelte-check found 0 errors and 2 warnings in 2 files

[YimingIsCOLD]

Any maintainers can help to review this?

[dummdidumm]

The ==== at the end helps me visually to separate the concrete errors from the summary. I would be ok with removing it in case there's no errors/warnings. The ==== at the start was for watch mode I believe, to separate the runs from each other.

@jasonlyu123 how do you feel about these tweaks?

[jasonlyu123]

I also feel similar. The ==== does make it easier to spot the separation. Removing the ==== in case there's no errors/warnings sounds good. Removing the multiple empty lines does make it more concise, so I think I'm okay with that.

[dummdidumm]

thank you!