postcss-normalize-charset 1.1.0 → 1.1.1

package/CHANGELOG.md

@@ -1,7 +1,13 @@
-# 1.1.0
-
-* Added `add` option (thanks to @ben-eb)
-
-# 1.0.0
-
-* Initial release.
+# 1.1.1
+
+* Performance tweaks; test that `node.parent` is equal to the AST rather than
+  checking its type is `root`, and use the AST directly to prepend the
+  `@charset` to, rather than using the superfluous `root()` method.
+
+# 1.1.0
+
+* Added `add` option (thanks to @ben-eb)
+
+# 1.0.0
+
+* Initial release.

package/LICENSE

@@ -1,20 +1,20 @@
-The MIT License (MIT)
-
-Copyright 2015 Bogdan Chadkin <trysound@yandex.ru>
-
-Permission is hereby granted, free of charge, to any person obtaining a copy of
-this software and associated documentation files (the "Software"), to deal in
-the Software without restriction, including without limitation the rights to
-use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
-the Software, and to permit persons to whom the Software is furnished to do so,
-subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
-FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
-COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
-IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
-CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+The MIT License (MIT)
+
+Copyright 2015 Bogdan Chadkin <trysound@yandex.ru>
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of
+this software and associated documentation files (the "Software"), to deal in
+the Software without restriction, including without limitation the rights to
+use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
+the Software, and to permit persons to whom the Software is furnished to do so,
+subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
+FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
+COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
+IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

package/README.md

@@ -1,44 +1,44 @@
-# postcss-normalize-charset [![Build Status][ci-img]][ci]
-
-Add necessary or remove extra charset with PostCSS
-
-```css
-a{
-  content: "©";
-}
-```
-
-```css
-@charset "utf-8";
-a{
-  content: "©";
-}
-```
-
-## API
-
-### normalizeCharset([options])
-
-#### options
-
-##### add
-
-Type: `boolean`
-Default: `true`
-
-Pass `false` to stop the module from adding a `@charset` declaration if it was
-missing from the file (and the file contained non-ascii characters).
-
-## Usage
-
-```js
-postcss([ require('postcss-normalize-charset') ])
-```
-
-See [PostCSS] docs for examples for your environment.
-
-MIT © [Bogdan Chadkin](mailto:trysound@yandex.ru)
-
-[PostCSS]: https://github.com/postcss/postcss
-[ci-img]:  https://travis-ci.org/TrySound/postcss-normalize-charset.svg
-[ci]:      https://travis-ci.org/TrySound/postcss-normalize-charset
+# postcss-normalize-charset [![Build Status][ci-img]][ci]
+
+Add necessary or remove extra charset with PostCSS
+
+```css
+a{
+  content: "©";
+}
+```
+
+```css
+@charset "utf-8";
+a{
+  content: "©";
+}
+```
+
+## API
+
+### normalizeCharset([options])
+
+#### options
+
+##### add
+
+Type: `boolean`
+Default: `true`
+
+Pass `false` to stop the module from adding a `@charset` declaration if it was
+missing from the file (and the file contained non-ascii characters).
+
+## Usage
+
+```js
+postcss([ require('postcss-normalize-charset') ])
+```
+
+See [PostCSS] docs for examples for your environment.
+
+MIT © [Bogdan Chadkin](mailto:trysound@yandex.ru)
+
+[PostCSS]: https://github.com/postcss/postcss
+[ci-img]:  https://travis-ci.org/ben-eb/postcss-normalize-charset.svg
+[ci]:      https://travis-ci.org/ben-eb/postcss-normalize-charset

package/index.js

@@ -1,35 +1,37 @@
-var postcss = require('postcss');
-
-module.exports = postcss.plugin('postcss-normalize-charset', function (opts) {
-    opts = opts || {};
-
-    return function (css) {
-        var charsetRule;
-        var nonAsciiNode;
-        var nonAscii = /[^\x00-\x7F]/;
-
-        css.walk(function (node) {
-            if (node.type === 'atrule' && node.name === 'charset') {
-                if (!charsetRule) {
-                    charsetRule = node;
-                }
-                node.remove();
-            } else if (!nonAsciiNode && node.parent && node.parent.type === 'root' && nonAscii.test(node)) {
-                nonAsciiNode = node;
-            }
-        });
-
-        if (nonAsciiNode) {
-            if (!charsetRule && opts.add !== false) {
-                charsetRule = postcss.atRule({
-                    name: 'charset',
-                    params: '"utf-8"'
-                });
-            }
-            if (charsetRule) {
-                charsetRule.source = nonAsciiNode.source;
-                css.root().prepend(charsetRule);
-            }
-        }
-    };
-});
+var postcss = require('postcss');
+
+var charset = 'charset';
+
+module.exports = postcss.plugin('postcss-normalize-' + charset, function (opts) {
+    opts = opts || {};
+
+    return function (css) {
+        var charsetRule;
+        var nonAsciiNode;
+        var nonAscii = /[^\x00-\x7F]/;
+
+        css.walk(function (node) {
+            if (node.type === 'atrule' && node.name === charset) {
+                if (!charsetRule) {
+                    charsetRule = node;
+                }
+                node.remove();
+            } else if (!nonAsciiNode && node.parent === css && nonAscii.test(node)) {
+                nonAsciiNode = node;
+            }
+        });
+
+        if (nonAsciiNode) {
+            if (!charsetRule && opts.add !== false) {
+                charsetRule = postcss.atRule({
+                    name: charset,
+                    params: '"utf-8"'
+                });
+            }
+            if (charsetRule) {
+                charsetRule.source = nonAsciiNode.source;
+                css.prepend(charsetRule);
+            }
+        }
+    };
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "postcss-normalize-charset",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "Add necessary or remove extra charset with PostCSS",
   "keywords": [
     "postcss",
@@ -13,20 +13,20 @@
     "index.js"
   ],
   "license": "MIT",
-  "repository": "TrySound/postcss-charset",
+  "repository": "ben-eb/postcss-charset",
   "bugs": {
-    "url": "https://github.com/TrySound/postcss-charset/issues"
+    "url": "https://github.com/ben-eb/postcss-charset/issues"
   },
-  "homepage": "https://github.com/TrySound/postcss-charset",
+  "homepage": "https://github.com/ben-eb/postcss-charset",
   "dependencies": {
     "postcss": "^5.0.5"
   },
   "devDependencies": {
+    "ava": "^0.16.0",
     "eslint": "^1.4.1",
-    "tap-spec": "^4.1.0",
-    "tape": "^4.2.0"
+    "postcss-devtools": "^1.1.1"
   },
   "scripts": {
-    "test": "eslint index.js test && tape test/*.js | tap-spec"
+    "test": "eslint index.js test && ava"
   }
 }