Skip to content

Migrate faker imports #1983

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

Merged
merged 2 commits into from
Dec 18, 2019
Merged

Migrate faker imports #1983

merged 2 commits into from
Dec 18, 2019

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Dec 18, 2019

ember-cli-mirage v1.0.0 removed its faker dependency in favor of importing via ember-auto-import. This PR adds an explicit dependency on faker, and migrates all the existing import to import from the 'faker' module directly.

@rust-highfive
Copy link

r? @jtgeibel

(rust_highfive has picked a reviewer for you, use r? to override)

@locks
Copy link
Contributor

locks commented Dec 18, 2019

Looks good!

@bors r+

@bors
Copy link
Contributor

bors commented Dec 18, 2019

📌 Commit c106410 has been approved by locks

@Turbo87
Copy link
Member Author

Turbo87 commented Dec 18, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Dec 18, 2019

📌 Commit ae3be09 has been approved by Turbo87

@bors
Copy link
Contributor

bors commented Dec 18, 2019

⌛ Testing commit ae3be09 with merge 2be7b7b...

bors added a commit that referenced this pull request Dec 18, 2019
Migrate `faker` imports

`ember-cli-mirage` v1.0.0 removed its `faker` dependency in favor of importing via `ember-auto-import`. This PR adds an explicit dependency on `faker`, and migrates all the existing import to import from the `'faker'` module directly.
@bors
Copy link
Contributor

bors commented Dec 18, 2019

☀️ Test successful - checks-travis
Approved by: Turbo87
Pushing 2be7b7b to master...

@bors bors merged commit ae3be09 into rust-lang:master Dec 18, 2019
@Turbo87 Turbo87 deleted the faker branch December 18, 2019 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants