Import the bundler into the monorepo #70
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…decide to resolve Import features without documents later. (It's been talked about.)
Honor the analyzer's resolved urls (present on the analyzer's documents, not import features)
…/polymer-bundler into going-all-in-on-polymer-analyzer
Going all in on polymer analyzer
Prepare to release pre.7
Bumped dependency on analyzer. Updated CHANGELOG.
When enabled, create identity sourcemaps or use existing sourcemaps during bundling. Each inline script will contain its own sourcemap.
This reverts commit cd0a196.
Add sourcemap support
* Updated CHANGELOG for 3.1.1 release. * Updated package-lock for 3.1.1 release.
* Switch to analyzer 3.0 (#617) * Updated polymer-analyzer to 3.0.0-pre.2 and fixed typing mismatches for string/ResolvedUrl etc. * Updated comments for ResolveUrl imports and put back the Document imports where they were. * Updated package.json and package-lock.json to indicate this is a 4.0.0-pre.x version.
* Removed UrlString as a thing. * Updated CHANGELOG to mention ResolvedUrl everywhere. * Make shards test helper functions more reasonable. * Added tests for the manifest in a redirect scenario for polymer-bundler binary. * Some semantic clean-up in the url-utils file. * Added comment about upcoming refactor to delegate relativeUrl calls to analyzer's UrlResolver. * Added many comments to the updated polymer-bundler.ts * Lets globally bump the mocha timeout to 5 seconds. * Created new url-utils to DRY up path resolution trailing slash logic and file URI production (getFileUrl and resolvePath) * Clean up a few more bits of polymer-bundler.ts * Fixed "url" to "URL" in comments. * Dried up url resolution by delegating to analyzer. * Updated typings and comment for ensureTrailingSlash so that it can spit out same type as it takes in. * Fix issue with temp dirs in polymer-bundler_test.ts * Updated to polymer-analyzer-3.0.0-pre.10
…ve URLs now. (#627) * Updated CHANGELOG. * Removed url-utils relativeUrl function. This function is the analyzer url resolvers' job now. * Updated bundler.ts to use the new resolvePath() function in url-utils.
…t collection. (#628) * Updated CHANGELOG. * Include the serialized document `content` in the BundleResult document collection. * Removed unused parse5 import from polymer-bundler.ts
…lass (#634) * All the code related to HTML fragment bundling is now in a class called HtmlBundler. * Updated CHANGELOG to describe the refactoring's breaking changes. * Renamed ast-utils to parse5-utils and removed import-utils. * findInSet is now find and instead of Set, supports Iterable.
…bundles. (#635) * Added .type to Bundle class in preparation of JavaScript module type bundles. * Changed type to support 'html-fragment' and 'es6-module' * Updated tests for mergeBundles.
* Prepare CHANGELOG for 4.0.0-pre.1 release.
* Updated mergeMatchingBundles to segregate merge operations by type. * Refactored the deps index file a bit to deduplicate the getDependencies for imports. * Updated deps index per PR feedback so that SubBundle URLs for external modules include the relative path to the module from the bundle entrypoint. * Infer bundle type from entrypoint URLs. * Updated CHANGELOG to remove the note about single file bundles. * Encapsulated knowledge of the `>` sub-bundle join characters into the deps-index file. * Updated the sub bundle naming scheme so the bundle type is reflected in the url. * Now with html-bundler inlining and rollup of bundles. * reservedBundleModuleExportNames more efficiently applied to all es6-module bundles via same analysis. * Convert some tests in deps-index_test.ts to inMemoryAnalyzer * Updated the rollup code to use Analyzer URL resolutions for bare module specifiers. * Updated CLI to support node module resolution flag. * Up the mocha timeout to 10 seconds because travis slow... * Added @types/chokidar after previously thinking that didnt exist. * Move @types/chokidar to devDependencies. * Moved some static file asset test fixtures into html-bundler_test.ts. * Updated test-utils comments and added tests for the in memory analyzer and heredoc. * Switched to getAnalysisDocument which throws error if not found. - PR feedback Polymer/polymer-bundler#637 (comment) * Exported functions from the bundler types instead of using the single-use bundler classes. * Got rid of the Es6ModuleBundler class in favor of a simple function. * Refactoring of the `_inlineScripts/_rollupInlineScripts` - Renamed `_inlineScripts` to `_inlineNonModuleScripts` - Renamed `_rollupInlineScripts` to `_inlineModuleScripts` - Moved responsibility for re-analyzing AST inside `_inlineModuleScripts` since that is implementation detail of that algorithm needing a `document.getFeatures` call. - Moved `rewriteNode` from `babel-utils` to `overwriteObject` in `utils.ts` since it is now used in two places. * Renamed `makeAbsolutePath` -> `ensureLeadingSlash`. - PR feedback Polymer/polymer-bundler#637 (comment) * Updated CHANGELOG and changed --in-html/--out-html to --in-file/--out-file. * Correctly modify the rollup document URL to be of .js extension.
….18 (#648) * Fixed explicit undefined checks while upgrading to analyzer 3.0.0-pre.18 * Little more refactoring for readability and brevity.
* Use Analyzer's 'export' feature scanner instead of babel traversing AST to obtain exported names. * Added test to demonstrate `export * from` * Updated CHANGELOG wrt export * from * Added test around export from to show current behavior.
* Updated CHANGELOG for 4.0.0-pre.3 release (with JS bundling!) * Updated package-lock.json for release.
aomarks
approved these changes
Apr 7, 2018
justinfagnani
added a commit
that referenced
this pull request
Jul 10, 2018
Simplify the interface between document converter and analysis converter
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.