chore(release): 4.0.0-rc.2

.travis.yml

@@ -5,9 +5,4 @@ node_js:
   - "12"
   - "14"
 
-script: yarn ci
-
-after_success:
-  - cat ./coverage/lcov.info | node_modules/.bin/coveralls --verbose
-  - cat ./coverage/coverage.json | node_modules/codecov.io/bin/codecov.io.js
-  - rm -rf ./coverage
+script: yarn test && coveralls < coverage/lcov.info

CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## [4.0.0-rc.2](https://github.com/postcss-modules-local-by-default/compare/v4.0.0-rc.1...v4.0.0-rc.2) - 2020-09-22
+
+### Fixes
+
+- avoid using `postcss` directly to create decls and rules
+
 ## [4.0.0-rc.1](https://github.com/postcss-modules-local-by-default/compare/v4.0.0-rc.0...v4.0.0-rc.1) - 2020-09-22
 
 ### BREAKING CHANGE

README.md

@@ -65,10 +65,6 @@ Note also you can _import_ multiple values at once but can only _define_ one val
 @value a: b, c: d; /* defines a as "b, c: d" */
 ```
 
-### Justification
-
-See [this PR](https://github.com/css-modules/css-modules-loader-core/pull/28) for more background
-
 ## License
 
 ISC

husky.config.js

@@ -0,0 +1,5 @@
+module.exports = {
+  hooks: {
+    "pre-commit": "lint-staged",
+  },
+};

lint-staged.config.js

@@ -0,0 +1,4 @@
+module.exports = {
+  "*.js": ["eslint --fix", "prettier --write"],
+  "*.{json,md,yml,css,ts}": ["prettier --write"],
+};

package.json

@@ -1,6 +1,6 @@
 {
   "name": "postcss-modules-values",
-  "version": "4.0.0-rc.1",
+  "version": "4.0.0-rc.2",
   "description": "PostCSS plugin for CSS Modules to pass arbitrary values between your module files",
   "main": "src/index.js",
   "files": [
@@ -13,12 +13,11 @@
     "prettier": "prettier -l --ignore-path .gitignore .",
     "eslint": "eslint --ignore-path .gitignore .",
     "lint": "yarn eslint && yarn prettier",
-    "pretest": "yarn lint",
-    "test": "jest",
     "test:only": "jest",
     "test:watch": "jest --watch",
     "test:coverage": "jest --coverage --collectCoverageFrom=\"src/**/*\"",
-    "test:ci": "yarn pretest && yarn cover",
+    "pretest": "yarn lint",
+    "test": "yarn test:coverage",
     "prepublishOnly": "yarn test"
   },
   "repository": {
@@ -37,8 +36,11 @@
   },
   "homepage": "https://github.com/css-modules/postcss-modules-values#readme",
   "devDependencies": {
+    "coveralls": "^3.1.0",
     "eslint": "^7.9.0",
+    "husky": "^4.3.0",
     "jest": "^26.4.2",
+    "lint-staged": "^10.4.0",
     "postcss": "^8.0.7",
     "prettier": "^2.1.2"
   },

src/index.js

@@ -1,6 +1,5 @@
 "use strict";
 
-const postcss = require("postcss");
 const ICSSUtils = require("icss-utils");
 
 const matchImports = /^(.+?|\([\s\S]+?\))\s+from\s+("[^"]*"|'[^']*'|[\w-]+)$/;
@@ -79,7 +78,7 @@ module.exports = () => {
             }
           },
         },
-        RootExit(root) {
+        RootExit(root, postcss) {
           /* We want to export anything defined by now, but don't add it to the CSS yet or it well get picked up by the replacement stuff */
           const exportDeclarations = Object.keys(definitions).map((key) =>
             postcss.decl({