Output list instead of table

[fregante]

• Fixes Output as list, not table #5423

Before

This isn't even the full table, it doesn't fit on my screen

After

[fregante]

@rpl @willdurand I believe the improvements here are evident from my screenshots:

• the output is shorter
• there's no duplicate explanation
  
• there's no bad wrapping of long text
  
• paths are always clickable
  
• row and col are part of the path, meaning IDEs can reach them in the same click
  
• the table implementation isn't even right because the error/warning tables are not aligned
  

The only reason not to move forward with this is because it already works, albeit in this suboptimal fashion. Since I'm currently available to work on this, allow me to complete this change and merge this PR. I don’t really like keeping PRs open for years

[Rob--W]

Thanks for improving the readability and dropping one dependency. We're willing to switch the output format.

Could you adjust the text and update the tests so that they pass?

Thanks for this, and apologies for the review delay!

[Rob--W]

Could you split this string construction in separate variables, so that there are no three ternary expressions embedded in one?

For example something like the following (untested, but just to show how it could be unrolled into something that is a bit more readable):

Suggested change

	const location = message.file
	? `${message.file}${message.line ? `:${message.line}` : ''}${
	message.column ? `:${message.column}` : ''
	}`
	: 'N/A';
	let location = message.file || 'N/A';
	if (message.file && message.line) {
	location += `:${message.line}`;
	if (message.column) {
	location += `:${message.column}`;
	}
	}

[fregante]

Great, let's see if I have time this month