@ui5/builder - npm Package Compare versions

Comparing version 0.0.2 to 0.1.0

.chglog/RELEASE.tpl.md

@@ -27,9 +27,8 @@ {{ range .Versions }}
 ### All changes
-<a name="{{ .Tag.Previous.Name }}...{{ .Tag.Name }}"></a>
-`[{{ .Tag.Previous.Name }}...{{ .Tag.Name }}]``
+[`{{ .Tag.Previous.Name }}...{{ .Tag.Name }}`]
 {{ end }}
 
 {{ if .Tag.Previous -}}
-[{{ .Tag.Previous.Name }}...{{ .Tag.Name }}]: {{ $.Info.RepositoryURL }}/compare/{{ .Tag.Previous.Name }}...{{ .Tag.Name }}
+[`{{ .Tag.Previous.Name }}...{{ .Tag.Name }}`]: {{ $.Info.RepositoryURL }}/compare/{{ .Tag.Previous.Name }}...{{ .Tag.Name }}
 {{ end -}}
 {{ end -}}

CHANGELOG.md

@@ -5,9 +5,56 @@ # Changelog
 
-A list of unreleased changes can be found [here](https://github.com/SAP/ui5-builder/compare/v0.0.2...HEAD).
+A list of unreleased changes can be found [here](https://github.com/SAP/ui5-builder/compare/v0.1.0...HEAD).
 
+<a name="v0.1.0"></a>
+## [v0.1.0] - 2018-06-26
+### Bug Fixes
+- Bundles should be built one after another [`164ba32`](https://github.com/SAP/ui5-builder/commit/164ba328c6e172297d71b9d3ef871005931cca71)
+- reduce build time at the price of a slightly increased code size ([#37](https://github.com/SAP/ui5-builder/issues/37)) [`1fb8d00`](https://github.com/SAP/ui5-builder/commit/1fb8d0049235467fcbd40f53e725cc419a8bc730)
+- Use the target bundle format to decide decoration ([#24](https://github.com/SAP/ui5-builder/issues/24)) [`83703bc`](https://github.com/SAP/ui5-builder/commit/83703bca17fd18b9ac700fae4801d87a4d86961d)
+
+### Internal Changes
+- Update ui5-fs and ui5-logger dependency [`dbc64df`](https://github.com/SAP/ui5-builder/commit/dbc64df8a67dd6c8d24704c45f6585ab1be97397)
+- Minor improvements around logging and error reporting ([#29](https://github.com/SAP/ui5-builder/issues/29)) [`55b1e89`](https://github.com/SAP/ui5-builder/commit/55b1e89b779b367db8aaa286e44a4f2c60ed074e)
+- Fix Changelog [`98612d4`](https://github.com/SAP/ui5-builder/commit/98612d410bd73976c8493692e11d12cd72ffbf5b)
+- **CHANGELOG:** Fix GitHub release template [`4b74ec5`](https://github.com/SAP/ui5-builder/commit/4b74ec5b7088d583de867a3bb9c116c634225ca2)
+
+
 <a name="v0.0.2"></a>
-## v0.0.2 - 2018-06-20
+## [v0.0.2] - 2018-06-21
 ### Bug Fixes
 - **Builders:** Do not bundle debug files [`19800a1`](https://github.com/SAP/ui5-builder/commit/19800a16689210c13495bc1bd0949896500cfc52)
 
+### Internal Changes
+- Add coveralls [`434b675`](https://github.com/SAP/ui5-builder/commit/434b67512444f279288359bf990895b607254075)
+- Add and update badges [`9c33b04`](https://github.com/SAP/ui5-builder/commit/9c33b047d9211e59f23e5fcbc76e66e5fd143150)
+- Have greenkeeper-lockfile amend any greenkeeper commit [`641817b`](https://github.com/SAP/ui5-builder/commit/641817b4e4f618aab6bbe3bde55ae01a942b93f8)
+- Prepare Greenkeeper installation [`fe9cbbf`](https://github.com/SAP/ui5-builder/commit/fe9cbbf0fbe3dd5bd8748adece8137797ae46795)
+- **Greenkeeper:** Add badge [`167cca9`](https://github.com/SAP/ui5-builder/commit/167cca9d038b4403fd282897b353db796fc0f7d9)
+- **Greenkeeper:** Add config file [`891339e`](https://github.com/SAP/ui5-builder/commit/891339e9493e8d0e609483891c476dea9c041d4e)
+- **README:** Pre-Alpha -> Alpha [`33c8190`](https://github.com/SAP/ui5-builder/commit/33c81906358d38b634099acbf8aafa234cc504db)
 
+
+<a name="v0.0.1"></a>
+## v0.0.1 - 2018-06-06
+### Bug Fixes
+- Restore default component preload [`a09bec2`](https://github.com/SAP/ui5-builder/commit/a09bec2f57f45a1c5d74681b3bdec4f7fdc45343)
+
+### Features
+- Add ability to configure component preloads and custom bundles [`2241e5f`](https://github.com/SAP/ui5-builder/commit/2241e5ff98fd95f1f80cc74959655ae7a9c660e7)
+
+### Internal Changes
+- Prepare npm release [`8947863`](https://github.com/SAP/ui5-builder/commit/8947863f6339d34aff801679e0338fe32c042194)
+- Update .editorconfig [`1f66211`](https://github.com/SAP/ui5-builder/commit/1f66211e3f7b82085caf90c341cee2c4c671fb8a)
+- Add chglog config + npm release scripts [`c82dc4e`](https://github.com/SAP/ui5-builder/commit/c82dc4e52c95260ba6e2c2f6423ce18ba9330267)
+- Update sap-ui-core bundle excludes [`d9d7a7a`](https://github.com/SAP/ui5-builder/commit/d9d7a7a75711c8f797c479dbd60b7c7aa2d984ea)
+- Update dependencies [`f852eb8`](https://github.com/SAP/ui5-builder/commit/f852eb87f98e0f1feb18fbe2b0306781f8ae52f1)
+- Add travis CI badge [`d34a3b2`](https://github.com/SAP/ui5-builder/commit/d34a3b264006dfacbd31cbb5ed2ef929fa8076b5)
+- Fix links to CONTRIBUTING.md file [`a3cc348`](https://github.com/SAP/ui5-builder/commit/a3cc3482cbb8c88b6e3dce6d46143473a66ce3e0)
+- **ESLint:** Activate no-console [`f8bd991`](https://github.com/SAP/ui5-builder/commit/f8bd99159c5359edf7bb53425c1650ee46fa0663)
+- **ESLint:** Activate no-var rule [`88c7950`](https://github.com/SAP/ui5-builder/commit/88c79501c3db9b579521a88d57a0e8a2742088bb)
+- **Readme:** Fix urls under Type and Processor section [`d1794bd`](https://github.com/SAP/ui5-builder/commit/d1794bd7026a9008b0b67870c91141f66511877a)
+- **Travis:** Add node.js 10 to test matrix [`ce91dd1`](https://github.com/SAP/ui5-builder/commit/ce91dd17e4e28932a838ec743a489ff6495d21a9)
+
+
+[v0.1.0]: https://github.com/SAP/ui5-builder/compare/v0.0.2...v0.1.0
+[v0.0.2]: https://github.com/SAP/ui5-builder/compare/v0.0.1...v0.0.2

lib/lbt/analyzer/ComponentAnalyzer.js

@@ -24,3 +24,3 @@ /**
 const ModuleName = require("../utils/ModuleName");
-const log = require("@ui5/logger").getLogger("ComponentAnalyzer");
+const log = require("@ui5/logger").getLogger("lbt:analyzer:ComponentAnalyzer");
 
@@ -55,5 +55,9 @@ // ---------------------------------------------------------------------------------------------------------
 		try {
-			let manifestResource = await this._pool.findResource(manifestName);
-			let fileContent = await manifestResource.buffer();
-			this._analyzeManifest( JSON.parse(fileContent.toString()), info );
+			let manifestResource = await this._pool.findResource(manifestName).catch(() => null);
+			if ( manifestResource ) {
+				let fileContent = await manifestResource.buffer();
+				this._analyzeManifest( JSON.parse(fileContent.toString()), info );
+			} else {
+				log.verbose("No manifest found for '%s', skipping analysis", resource.name);
+			}
 		} catch (err) {
@@ -60,0 +64,0 @@ log.error("an error occurred while analyzing component %s (ignored)", resource.name, err);

lib/lbt/analyzer/FioriElementsAnalyzer.js

@@ -66,3 +66,3 @@ /**
 const {getValue, isMethodCall, isString} = require("../utils/ASTUtils");
-const log = require("@ui5/logger").getLogger("FioriElementAnalyzer");
+const log = require("@ui5/logger").getLogger("lbt:analyzer:FioriElementAnalyzer");
 
@@ -99,5 +99,9 @@ // ---------------------------------------------------------------------------------------------------------
 		try {
-			let manifestResource = await this._pool.findResource(manifestName);
-			let fileContent = await manifestResource.buffer();
-			await this._analyzeManifest( JSON.parse(fileContent.toString()), info );
+			let manifestResource = await this._pool.findResource(manifestName).catch(() => null);
+			if ( manifestResource ) {
+				let fileContent = await manifestResource.buffer();
+				await this._analyzeManifest( JSON.parse(fileContent.toString()), info );
+			} else {
+				log.verbose("No manifest found for '%s', skipping analysis", resource.name);
+			}
 		} catch (err) {
@@ -104,0 +108,0 @@ log.error("an error occurred while analyzing template app %s (ignored)", resource.name, err);

lib/lbt/analyzer/JSModuleAnalyzer.js

@@ -10,3 +10,3 @@ "use strict";
 const {findOwnProperty, getLocation, getPropertyKey, isMethodCall, isString} = require("../utils/ASTUtils");
-const log = require("@ui5/logger").getLogger("JSModuleAnalyzer");
+const log = require("@ui5/logger").getLogger("lbt:analyzer:JSModuleAnalyzer");
 
@@ -390,3 +390,3 @@ // --------------------------------------------------------------------------------------------------------------------
 			if ( nArgs > 0 && args[0].type == Syntax.OBJECT ) {
-				log.warn("jQuery.sap.require: cannot evaluate complex require (view/controller)");
+				log.verbose("jQuery.sap.require: cannot evaluate complex require (view/controller)");
 			} else {
@@ -406,3 +406,3 @@ // UI5 signature with one or many required modules
 					} else {
-						log.warn("jQuery.sap.require: cannot evaluate dynamic arguments: ", arg);
+						log.verbose("jQuery.sap.require: cannot evaluate dynamic arguments: ", arg);
 						info.dynamicDependencies = true;
@@ -409,0 +409,0 @@ }

lib/lbt/analyzer/SmartTemplateAnalyzer.js

@@ -33,3 +33,3 @@ /**
 const {getValue, isMethodCall, isString} = require("../utils/ASTUtils");
-const log = require("@ui5/logger").getLogger("SmartTemplateAnalyzer");
+const log = require("@ui5/logger").getLogger("lbt:analyzer:SmartTemplateAnalyzer");
 
@@ -58,5 +58,9 @@ // ---------------------------------------------------------------------------------------------------------
 		try {
-			let manifestResource = await this._pool.findResource(manifestName);
-			let fileContent = await manifestResource.buffer();
-			await this._analyzeManifest( JSON.parse(fileContent.toString()), info );
+			let manifestResource = await this._pool.findResource(manifestName).catch(() => null);
+			if ( manifestResource ) {
+				let fileContent = await manifestResource.buffer();
+				await this._analyzeManifest( JSON.parse(fileContent.toString()), info );
+			} else {
+				log.verbose("No manifest found for '%s', skipping analysis", resource.name);
+			}
 		} catch (err) {
@@ -63,0 +67,0 @@ log.error("an error occurred while analyzing template app %s (ignored)", resource.name, err);

lib/lbt/analyzer/XMLCompositeAnalyzer.js

@@ -8,3 +8,3 @@ "use strict";
 const ModuleName = require("../utils/ModuleName");
-const log = require("@ui5/logger").getLogger("XMLCompositeAnalyzer");
+const log = require("@ui5/logger").getLogger("lbt:analyzer:XMLCompositeAnalyzer");
 
@@ -58,3 +58,3 @@ const CALL_DEFINE = ["define"];
 	_analyzeXMLCClassDefinition(clazz) {
-		log.verbose(clazz);
+		// log.verbose(clazz);
 		let fragmentName = getValue(clazz, ["fragment"]);
@@ -61,0 +61,0 @@ if ( isString(fragmentName) ) {

lib/lbt/analyzer/XMLTemplateAnalyzer.js

@@ -15,3 +15,3 @@ "use strict";
 const ModuleName = require("../utils/ModuleName");
-const log = require("@ui5/logger").getLogger("XMLTemplateAnalyzer");
+const log = require("@ui5/logger").getLogger("lbt:analyzer:XMLTemplateAnalyzer");
 
@@ -18,0 +18,0 @@ // ---------------------------------------------------------------------------------------------------------

lib/lbt/bundle/AutoSplitter.js

@@ -7,3 +7,3 @@ "use strict";
 const {SectionType} = require("./BundleDefinition");
-const log = require("@ui5/logger").getLogger("AutoSplitter");
+const log = require("@ui5/logger").getLogger("lbt:bundle:AutoSplitter");
 
@@ -10,0 +10,0 @@ const copyrightCommentsPattern = /copyright|\(c\)|released under|license|\u00a9/i;

lib/lbt/bundle/Builder.js

@@ -19,3 +19,3 @@ /* eslint quotes: ["error", "double", { "allowTemplateLiterals": true }] */
 const BundleWriter = require("./BundleWriter");
-const log = require("@ui5/logger").getLogger("Builder");
+const log = require("@ui5/logger").getLogger("lbt:bundle:Builder");
 
@@ -140,3 +140,4 @@ const copyrightCommentsPattern = /copyright|\(c\)|released under|license|\u00a9/i;
 		this.shouldDecorate = this.options.decorateBootstrapModule &&
-			(((this.optimizedSources && !this.options.debugMode) || this.optimize) && resolvedModule.containsGlobal);
+			(((this.optimizedSources && !this.options.debugMode) || this.optimize) &&
+				this.targetBundleFormat.shouldDecorate(resolvedModule));
 		// TODO is the following condition ok or should the availability of jquery.sap.global.js be configurable?
@@ -143,0 +144,0 @@ this.jqglobalAvailable = !resolvedModule.containsGlobal;

lib/lbt/bundle/Resolver.js

@@ -12,3 +12,3 @@ /**
 const ResolvedBundleDefinition = require("./ResolvedBundleDefinition");
-const log = require("@ui5/logger").getLogger("Resolver");
+const log = require("@ui5/logger").getLogger("lbt:bundle:Resolver");
 
@@ -15,0 +15,0 @@ let dependencyTracker;

lib/lbt/graph/dependencyGraph.js

 "use strict";
 
-const log = require("@ui5/logger").getLogger("dependencyGraph");
+const log = require("@ui5/logger").getLogger("lbt:graph:dependencyGraph");
 
@@ -5,0 +5,0 @@ class Node {

lib/lbt/graph/dominatorTree.js

@@ -19,3 +19,3 @@ "use strict";
 function calculateDominatorTree({n0, nodes}) {
-	const log = require("@ui5/logger").getLogger("dominatorTree");
+	const log = require("@ui5/logger").getLogger("lbt:graph:dominatorTree");
 	// initialize set of dominators for each node
@@ -22,0 +22,0 @@ for ( let n of nodes.values() ) {

lib/lbt/graph/topologicalSort.js

 "use strict";
 
-const log = require("@ui5/logger").getLogger("topologicalSort");
+const log = require("@ui5/logger").getLogger("lbt:graph:topologicalSort");
 
@@ -5,0 +5,0 @@ /**

lib/lbt/resources/ResourceFilterList.js

 "use strict";
 
-const log = require("@ui5/logger").getLogger("ResourceFilterList");
+const log = require("@ui5/logger").getLogger("lbt:resources:ResourceFilterList");
 
@@ -5,0 +5,0 @@ function makeMatcher(globPattern) {

lib/lbt/resources/ResourcePool.js

@@ -17,3 +17,3 @@ "use strict";
 const Resource = require("./Resource");
-const log = require("@ui5/logger").getLogger("ResourcePool");
+const log = require("@ui5/logger").getLogger("lbt:resources:ResourcePool");
 
@@ -20,0 +20,0 @@ const jsAnalyzer = new JSModuleAnalyzer();

lib/lbt/utils/ModuleName.js

@@ -101,6 +101,3 @@ "use strict";
 					if ( segments.length === 0 ) {
-						throw new Error(String.format("Can't navigate to parent of root (%s", relativePath));
-						// NODE-TODO, getPackagePath not defined:
-						//	throw new Error(String.format(
-						//		"Can't navigate to parent of root (%s %s", getPackagePath(), relativePath));
+						throw new Error(`Can't navigate to parent of root (${path}, ${relativePath})`);
 					}
@@ -111,3 +108,3 @@ segments.pop();
 					// segment '...' or more dots: not allowed
-					throw new Error(String.format("Illegal path segment '%s'", segment));
+					throw new Error(`Illegal path segment '${segment}'`);
 				}
@@ -114,0 +111,0 @@ } else {

lib/processors/uglifier.js

@@ -21,3 +21,4 @@ const uglify = require("uglify-es");
 					comments: copyrightCommentsPattern
-				}
+				},
+				compress: false
 			});
@@ -24,0 +25,0 @@ if (result.error) {

lib/types/library/LibraryBuilder.js

@@ -86,13 +86,15 @@ const AbstractBuilder = require("../AbstractBuilder");
 			this.addTask("generateBundle", () => {
-				return Promise.all(bundles.map((bundle) => {
-					return tasks.generateBundle({
-						workspace: resourceCollections.workspace,
-						dependencies: resourceCollections.dependencies,
-						options: {
-							projectName: project.metadata.name,
-							bundleDefinition: bundle.bundleDefinition,
-							bundleOptions: bundle.bundleOptions
-						}
+				return bundles.reduce(function(sequence, bundle) {
+					return sequence.then(function() {
+						return tasks.generateBundle({
+							workspace: resourceCollections.workspace,
+							dependencies: resourceCollections.dependencies,
+							options: {
+								projectName: project.metadata.name,
+								bundleDefinition: bundle.bundleDefinition,
+								bundleOptions: bundle.bundleOptions
+							}
+						});
 					});
-				}));
+				}, Promise.resolve());
 			});
@@ -99,0 +101,0 @@ }

package.json

 {
 	"name": "@ui5/builder",
-	"version": "0.0.2",
+	"version": "0.1.0",
 	"description": "UI5 Build and Development Tooling - Builder",
@@ -101,4 +101,4 @@ "author": "SAP SE (https://www.sap.com)",
 	"dependencies": {
-		"@ui5/fs": "^0.0.1",
-		"@ui5/logger": "^0.0.1",
+		"@ui5/fs": "^0.1.0",
+		"@ui5/logger": "^0.1.0",
 		"archiver": "^2.1.0",
@@ -105,0 +105,0 @@ "escope": "^3.6.0",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

308042

increased by1.52%

Lines of code

7559

increased by0.17%

Dependency changes

• + Added@ui5/fs@0.1.0(transitive)

• + Added@ui5/logger@0.1.0(transitive)

• - Removed@ui5/fs@0.0.1(transitive)

• - Removed@ui5/logger@0.0.1(transitive)

• Updated@ui5/fs@^0.1.0

• Updated@ui5/logger@^0.1.0