Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use new EsCard and EsLinkCard components #542

Merged
merged 4 commits into from
Feb 4, 2020
Merged

Conversation

pichfl
Copy link
Contributor

@pichfl pichfl commented Jan 31, 2020

Might have missed some of them, also depends on ember-learn/ember-styleguide#279 and an update of the style guide /cc @mansona

@pichfl pichfl requested a review from mansona January 31, 2020 17:56
@MelSumner
Copy link
Member

@pichfl can you update to resolve the merge conflicts please? thank you! 👍

@pichfl
Copy link
Contributor Author

pichfl commented Feb 2, 2020

Done.

It still requires a fresh release of the style guide, @mansona should be able to create one and add the change to this.

Might have missed some of them
@mansona
Copy link
Member

mansona commented Feb 4, 2020

I would be happier to merge ember-learn/ember-styleguide#280 before the next release of the styleguide 👍 happy to update and review this once that is done

@github-actions
Copy link

github-actions bot commented Feb 4, 2020

Files that got Bigger 🚨:

File raw gzip
vendor.js +11.1 kB +1.91 kB

Files that got Smaller 🎉:

File raw gzip
ember-website.js -509 B +28 B
vendor.css -481 B -65 B

Files that stayed the same size 🤷‍:

File raw gzip
auto-import-fastboot.js 0 B 0 B
ember-website-fastboot.js 0 B 0 B
ember-website.css 0 B 0 B

Copy link
Member

@mansona mansona left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@github-actions
Copy link

github-actions bot commented Feb 4, 2020

Files that got Bigger 🚨:

File raw gzip
vendor.js +11.1 kB +1.91 kB

Files that got Smaller 🎉:

File raw gzip
ember-website.js -537 B -5 B
vendor.css -481 B -65 B

Files that stayed the same size 🤷‍:

File raw gzip
auto-import-fastboot.js 0 B 0 B
ember-website-fastboot.js 0 B 0 B
ember-website.css 0 B 0 B

@mansona mansona merged commit 4e11347 into redesign-rfc Feb 4, 2020
@mansona mansona deleted the use-escard branch February 4, 2020 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants