Skip to content

docs: Dependency Injection guides for CLI - #19892

Closed
wardbell wants to merge 1 commit into
angular:masterfrom
IdeaBlade:clify-di
Closed

wardbell wants to merge 1 commit into
angular:masterfrom
IdeaBlade:clify-di

Conversation

@wardbell

Copy link
Copy Markdown
Contributor

Issue #19510

PR Checklist

Please check if your PR fulfills the following requirements:

@wardbell wardbell self-assigned this Oct 24, 2017
@wardbell
wardbell requested a review from Foxandxss October 24, 2017 09:48
@Foxandxss Foxandxss mentioned this pull request Oct 24, 2017
21 of 24 tasks
@Foxandxss Foxandxss added this to the 5.0.x milestone Nov 2, 2017
@mary-poppins

Copy link
Copy Markdown

You can preview a056241 at https://pr19892-a056241.ngbuilds.io/.

@vicb

vicb commented Nov 2, 2017

Copy link
Copy Markdown
Contributor

missing:

  • target label
  • merge label
  • green CI

@IgorMinar IgorMinar added the target: patch This PR is targeted for the next patch release label Nov 2, 2017

@IgorMinar IgorMinar left a comment

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.

lgtm for navigation.json changes. the ci is still failing so this can't be merged until that is resolved.

@mary-poppins

Copy link
Copy Markdown

You can preview 128a27b at https://pr19892-128a27b.ngbuilds.io/.

@mary-poppins

Copy link
Copy Markdown

You can preview 319ccfb at https://pr19892-319ccfb.ngbuilds.io/.

@mary-poppins

Copy link
Copy Markdown

You can preview 94eb9ce at https://pr19892-94eb9ce.ngbuilds.io/.

@Foxandxss Foxandxss added action: merge The PR is ready for merge by the caretaker and removed state: WIP labels Nov 13, 2017
@jasonaden jasonaden closed this in c031860 Nov 14, 2017
jasonaden pushed a commit that referenced this pull request Nov 14, 2017
@wardbell
wardbell deleted the clify-di branch November 16, 2017 06:40
@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 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants