Skip to content

[ember] add Application/Engine#buildInstance()#28114

Closed
jamescdavis wants to merge 1 commit intoDefinitelyTyped:masterfrom
jamescdavis:add_buildInstance_to_ember
Closed

[ember] add Application/Engine#buildInstance()#28114
jamescdavis wants to merge 1 commit intoDefinitelyTyped:masterfrom
jamescdavis:add_buildInstance_to_ember

Conversation

@jamescdavis
Copy link
Contributor

Please fill in this template.

  • Use a meaningful title for the pull request. Include the name of the package modified.
  • Test the change in your own code. (Compile and run.)
  • Add or edit tests to reflect the change. (Run with npm test.)
  • Follow the advice from the readme.
  • Avoid common mistakes.
  • Run npm run lint package-name (or tsc if no tslint.json is present).

Select one of these and delete the others:

If changing an existing definition:

@typescript-bot
Copy link
Contributor

typescript-bot commented Aug 14, 2018

@jamescdavis Thank you for submitting this PR!

🔔 @jedmao @bttf @dwickern @chriskrycho @theroncross @mfeckie @alexlafroscia @mike-north - please review this PR in the next few days. Be sure to explicitly select Approve or Request Changes in the GitHub UI so I know what's going on.

If no reviewer appears after a week, a DefinitelyTyped maintainer will review the PR instead.

@typescript-bot
Copy link
Contributor

typescript-bot commented Aug 14, 2018

@jamescdavis The Travis CI build failed! Please review the logs for more information.

Once you've pushed the fixes, the build will automatically re-run. Thanks!

@jamescdavis
Copy link
Contributor Author

jamescdavis commented Aug 14, 2018

Build failure appears to be a regression in [email protected]

@typescript-bot
Copy link
Contributor

@jamescdavis The Travis CI build failed! Please review the logs for more information. Once you've pushed the fixes, the build will automatically re-run. Thanks!

@typescript-bot typescript-bot added the Abandoned This PR had no activity for a long time, and is considered abandoned label Aug 21, 2018
@typescript-bot
Copy link
Contributor

@jamescdavis To keep things tidy, we have to close PRs that aren't mergeable but don't have activity from their author. No worries, though - please open a new PR if you'd like to continue with this change. Thank you!

@mike-north
Copy link
Contributor

A breaking change in TS 3.1 held up this PR. Should be good to review/merge after #28282 goes in

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Abandoned This PR had no activity for a long time, and is considered abandoned

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants