standard / snazzy

Format JavaScript Standard Style as Stylish (i.e. snazzy) output
https://standardjs.com
MIT License
422 stars 18 forks source link

Add warning support to Snazzy to allow Standard 17 #29

Open tagliala opened 2 years ago

tagliala commented 2 years ago

Do you know which package is in charge of building this json?

{
  line: '2',
  column: '17',
  message: 'Expected property shorthand.',
  ruleId: 'object-shorthand) (warning'
}

If this is not being created by Snazzy (as it appears to my eyes, I'm not a JavaScript developer), I think that there could be an issue somewhere else upstream that should be fixed before adding warning support to Snazzy

Originally posted by @tagliala in https://github.com/standard/eslint-config-standard/issues/229#issuecomment-1140282410

xChickens commented 1 year ago

@voxpelli Any chance you could take a look at this?

tagliala commented 1 year ago

Hello,

this requires a combination of 2 PRs:

32 needs to be adjusted by deciding what it is needed to do with warnings

image

I would be glad if you can continue my PRs. standard/standard-json#14 is also tested, but you may not like the implementation

tagliala commented 1 year ago

Friendly bump.

Since standard seems still supported, after the recent 17.1.0 release, it may be a good moment to also upgrade snazzy and standard-json

voxpelli commented 1 year ago

Since standard seems still supported, after the recent 17.1.0 release, it may be a good moment to also upgrade snazzy and standard-json

I did those releases as I still used those modules + found some time in my calendar.

I don’t use either of snazzy or standard-json and I’m not having time to help out maintaining standard much anymore, so you would have check with @feross