Skip to content

Commit

Permalink
v2.3.22
Browse files Browse the repository at this point in the history
- Fix riot/riot#1511 : Escape Quotes - They may be some issues to fix.
- Regression of logic to parse style and script tags, due to loss of performance and other issues.
- Removed the "compress" option of the `less` parser, which is deprecated and generates warnings in the console.
- Removed the unuseful CSS parser `stylus` from the browser version.
- Refactorization of all the code, with more comments in preparation for the automatic documentation of the API.
- Various tweaks to increase performance and reduce (~55%) memory consumption.
- Files to preprocess are moved from "lib" to the "src" directory, now "lib" has the required node.js files only.
  • Loading branch information
amarcruz committed Jan 24, 2016
2 parents 3e18c3d + 20661c0 commit 132d8af
Show file tree
Hide file tree
Showing 54 changed files with 3,561 additions and 2,480 deletions.
2 changes: 2 additions & 0 deletions .eslintignore
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,5 @@
**/test/specs/fixtures/
**/test/specs/parsers/js/
**/test/v223/
**/test/perf.js
**/src/
4 changes: 2 additions & 2 deletions .eslintrc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ rules:
# STYLISTIC ISSUES
#
array-bracket-spacing: 2 # disallow spaces inside of brackets (Default: [0, "never"])
brace-style: [1, "stroustrup", { "allowSingleLine": true }]
brace-style: [1, "1tbs", { "allowSingleLine": true }]
# require Stroustrup style: `else` must be on its own line (Default: [0, '1tbs'])
camelcase: [2, { "properties": "always" }]
# require camel case names (Default: 0)
Expand All @@ -130,7 +130,7 @@ rules:
# (Default: [0, { "beforeColon": false, "afterColon": true }])
linebreak-style: [2, "unix"]
max-depth: [1, 5] # specify the maximum depth blocks can be nested
max-len: [1, 92, 4, { "ignoreUrls": true, "ignorePattern": "=\\s+/.+/|_regEx\\(" }]
max-len: [1, 92, 4, { "ignoreUrls": true, "ignorePattern": "=\\s+/.+/|_regEx\\(|RegExp\\(" }]
# maximum line length, except for regexes
max-nested-callbacks: [2, 4] # specify the maximum depth callbacks can be nested (Default: [0, 2])
new-cap: 2 # require a capital letter for constructors (Default: 0)
Expand Down
12 changes: 9 additions & 3 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,15 @@
# Compiler Changes

### v2.3.21
- Code refactorization of the `parsers` module, removed unuseful CSS parser `stylus` for browsers.
### v2.3.22
- Fix [riot#1511](https://github.com/riot/riot/issues/1511) : Escape Quotes - They may be some issues to fix.
- New logic for parsing `<script>/<style>` blocks. See [doc/guide.md](https://github.com/riot/compiler/blob/dev/doc/guide.md#the-untagged-javascript-block) for details.
- Regression of logic to parse style and script tags, due to loss of performance and other issues.
- Removed the "compress" option of the `less` parser, which is deprecated and generates warnings in the console.
- Removed the unuseful CSS parser `stylus` from the browser version.
- Refactorization of all the code, with more comments in preparation for the automatic documentation of the API.
- Various tweaks to increase performance and reduce (~55%) memory consumption.
- Files to preprocess are moved from "lib" to the "src" directory, now "lib" has the required node.js files only.

### v2.3.21 (unpublished due to errors)

### v2.3.20
- Fix [riot#1495](https://github.com/riot/riot/issues/1495) : Warning of input tag value - Avoids warnings for date/datetime/time/month/email/color types with expression in its value.
Expand Down
21 changes: 13 additions & 8 deletions Makefile
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
# jspreproc flags
#JSPP_DEBUG = -D DEBUG
JSPP_FLAGS = -F istanbul -F eslint --custom-filter "\s@(module|version)\b" --headers ""
JSPP_RIOT_FLAGS = $(JSPP_FLAGS) -D RIOT
JSPP_NODE_FLAGS = $(JSPP_FLAGS) -D NODE
JSPP_NODE_FLAGS = $(JSPP_FLAGS) -D NODE -F jsdoc
JSPP_RIOT_FLAGS = $(JSPP_FLAGS)
JSPP_ES6_FLAGS = $(JSPP_FLAGS)

# Code Climate only accepts the first job of default branch
Expand All @@ -21,12 +20,15 @@ DIST = "./dist/"
# default job
test: build test-mocha

build: eslint
build: pre-build eslint
# rebuild all
@ $(JSPP) $(JSPP_RIOT_FLAGS) src/_riot.js > $(DIST)riot.compiler.js
@ $(JSPP) $(JSPP_ES6_FLAGS) src/_es6.js > $(DIST)es6.compiler.js

pre-build:
# build the node version
@ mkdir -p $(DIST)
@ $(JSPP) $(JSPP_RIOT_FLAGS) lib/_riot.js > $(DIST)riot.compiler.js
@ $(JSPP) $(JSPP_NODE_FLAGS) lib/_node.js > $(DIST)compiler.js
@ $(JSPP) $(JSPP_ES6_FLAGS) lib/_es6.js > $(DIST)es6.compiler.js
@ $(JSPP) $(JSPP_NODE_FLAGS) src/core.js > lib/compiler.js

eslint:
# check code style
Expand All @@ -51,4 +53,7 @@ debug: build
perf: build
@ node --expose-gc test/perf.js

.PHONY: test build eslint test-mocha send-coverage debug perf
docs: build
@ jsdoc lib/ --configure ./jsdoc.json -P ./package.json --verbose

.PHONY: test pre-build build eslint test-mocha send-coverage debug perf docs
Loading

0 comments on commit 132d8af

Please sign in to comment.