Skip to content
This repository has been archived by the owner on Dec 18, 2024. It is now read-only.

Consume pre-built examples from material2 repo #160

Merged
merged 2 commits into from
Jun 1, 2017

Conversation

jelbourn
Copy link
Member

@jelbourn jelbourn commented May 30, 2017

This PR:

  • Deletes src/app/examples
  • Deletes src/assets/plunker/examples
  • Changes imports to use @angular/material-examples/
  • Updates the fetch-assets.sh script to npm install the
    material-examples package from material2-docs-content.
  • Updates the fetch-assets.sh script to copy example sources from material2-docs-content into assets for plunkers

This PR:
* Deletes src/app/examples
* Changes imports to use `@angular/material-examples/`
* Updates the `fetch-assets.sh` script to `npm install` the
material-examples package from material2-docs-content.
Copy link
Member

@devversion devversion left a comment

Choose a reason for hiding this comment

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

LGTM - one typo

@@ -62,7 +60,8 @@ describe('ExampleViewer', () => {
component.example = exampleComponentString;
fixture.detectChanges();
const data = component.exampleData;
expect(data).toEqual(EXAMPLE_COMPONENTS[exampleComponentString]);
// TODO(jelbourn): remove `as any` once LiveExample is updated to hve optional members.
Copy link
Member

Choose a reason for hiding this comment

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

Typo for have

Copy link
Contributor

@tinayuangao tinayuangao left a comment

Choose a reason for hiding this comment

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

LGTM

@jelbourn
Copy link
Member Author

Updated PR to copy assets for plunker as well

@jelbourn jelbourn merged commit 727813b into angular:master Jun 1, 2017
@huxulm huxulm mentioned this pull request Oct 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants