X Tutup
Skip to content

feat(forms.ts): formInjectables with FormBuilder#2367

Closed
PatrickJS wants to merge 1 commit intoangular:masterfrom
PatrickJS:patch-4
Closed

feat(forms.ts): formInjectables with FormBuilder#2367
PatrickJS wants to merge 1 commit intoangular:masterfrom
PatrickJS:patch-4

Conversation

@PatrickJS
Copy link
Copy Markdown
Contributor

No description provided.

@vicb
Copy link
Copy Markdown
Contributor

vicb commented Jun 5, 2015

Please run clang-format.

@vicb vicb added the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label Jun 5, 2015
@PatrickJS
Copy link
Copy Markdown
Contributor Author

thanks @vicb, I saw the no_dart error and just assumed it was okay since it worked in Dart. I pushed it up with the correct format

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.

I don't think it will work in Dart. You should use: export const formInjectables: List<any> = CONST_EXPR([FormBuilder]);

@PatrickJS
Copy link
Copy Markdown
Contributor Author

this should be good for Dart now

@tbosch tbosch closed this in a6cb86b Jun 10, 2015
@PatrickJS PatrickJS deleted the patch-4 branch June 11, 2015 07:26
@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 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup