fix(options): disallow additional properties and add ident to validation

[fnatte]

Prevents unsupported options in webpack config.
Unknown properties will cause a compilation {Error}

Module build failed: ValidationError: PostCSS Loader Invalid Options

options['other'] is an invalid additional property

Type

---

• Fix

SemVer

---

• Bug (:label: Patch)

Checklist

---

• Lint and unit tests pass with my changes
• I have added tests that prove my fix is effective/works
• I have added necessary documentation (if appropriate)
• Any dependent changes are merged and published in downstream modules

[coveralls]

Coverage remained the same at 87.975% when pulling b90a561 on Fnatte:fix/update-options-json into 1e0cade on postcss:master.

[michael-ciniawsky]

@fnatte Thx

[michael-ciniawsky]

Released in v2.1.2 🎉

[koresar]

According to SemVer it should have been the v3.0.0 release. This broke many builds across the internet. Mostly Vue.js projects which include postcss by default.

Would you please revert this?

[alexander-akait]

/cc @michael-ciniawsky it is breaking, need revert and release new major version 👍

[koresar]

@ai Hi Adnrey. Whole Internet is shocked with this bug. Could you please revert the PR and publish v2.1.3 it? Looks like @michael-ciniawsky is missing somewhere.

[ai]

@koresar tell me more about the issue. I can remove the release from npm, but I need to clearly understand what happened to not make it worse.

[koresar]

Andrey, there are dozens of links linked to this github issues. Here is a full error. nuxt/nuxt#3048

…

On 20 Mar. 2018 12:28, "Andrey Sitnik" ***@***.***> wrote: @koresar <https://github.com/koresar> tell me more about the issue. I can remove the release from npm, but I need to clearly understand what happened to not make it worse. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#346 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABjCL-1AlDeVMPBN-q3as1XlOL6u4CHRks5tgFtOgaJpZM4Su9HP> .

[ai]

@koresar it is not a “tell me more” ;). How options['useConfigFile'] is an invalid additional property is connected with this PR?

[ai]

I reverted additionalProperties and released 2.1.3

@michael-ciniawsky sorry that I did it for you, emergency situation

[koresar]

Thank you @ai for the collaboration and dedication.