Skip to content
This repository was archived by the owner on Nov 22, 2024. It is now read-only.

feat(express-engine): add schematics#1051

Merged
CaerusKaru merged 13 commits intomasterfrom
adam/schematics
Aug 22, 2018
Merged

feat(express-engine): add schematics#1051
CaerusKaru merged 13 commits intomasterfrom
adam/schematics

Conversation

@CaerusKaru
Copy link
Copy Markdown
Member

@CaerusKaru CaerusKaru commented Aug 15, 2018

  • Adds schematics to the Express Engine that allow
    for easier installation. This is basically an
    enhanced version of the vanilla Universal schematic
    that is available by default, where this one adds
    the Express Engine, a sample Express server, as well
    as some other necessary dependencies
  • Credit to @devversion on the Material team for the
    entire Bazel setup

Fixes #968

@CaerusKaru CaerusKaru added state: WIP area: express-engine target: minor target: minor This PR is targeted for the next minor release labels Aug 15, 2018
@CaerusKaru CaerusKaru requested a review from vikerman as a code owner August 15, 2018 20:32
Copy link
Copy Markdown
Member Author

@CaerusKaru CaerusKaru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also TODO: add Bazel build steps and work out the linting errors that are bound to come up

### Install
Adds Angular Universal Express Engine and its depedencies and pre-configures the application.

- Adds Express-Engine, NgModule-Factory-Loader, ts-loader, and webpack to `package.json`
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will finalize docs when behavior is finalized. Part of this was stolen from Material

@@ -0,0 +1,179 @@
/**
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

More tests will be needed as behavior is finalized. These are the default tests, plus some added dependency checks. Will also need test runners that work with Bazel (Material has this already)

@CaerusKaru CaerusKaru changed the title WIP feat(express-engine): add schematics feat(express-engine): add schematics Aug 18, 2018
@@ -0,0 +1,41 @@
/**
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file can be removed. When it was taken from Material, they use it for their update task to run things sequentially. We don't, so we don't need it here.

const pkg = JSON.parse(buffer.toString());

pkg.dependencies['@nguniversal/express-engine'] = '0.0.0-PLACEHOLDER';
pkg.dependencies['@nguniversal/module-map-ngfactory-loader'] = '6.0.0' || '0.0.0-PLACEHOLDER';
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is the "||" needed? Isn't it a no-op?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a no-op, but it enables local testing using npm link. When built though, the linked version has a long version name that coincides with the commit SHA, but the NPM version for the other @nguniversal packages remain normal.

@CaerusKaru CaerusKaru merged commit 1909be1 into master Aug 22, 2018
@CaerusKaru CaerusKaru deleted the adam/schematics branch August 22, 2018 02:25
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Sep 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: review area: express-engine target: minor target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support schematics in @nguniversal/express-engine

4 participants