X Tutup
Skip to content

refactor(dart/transform): Migrates tests to use package:test#7475

Closed
kegluneq wants to merge 1 commit intoangular:masterfrom
kegluneq:package_test3
Closed

refactor(dart/transform): Migrates tests to use package:test#7475
kegluneq wants to merge 1 commit intoangular:masterfrom
kegluneq:package_test3

Conversation

@kegluneq
Copy link
Copy Markdown

@kegluneq kegluneq commented Mar 8, 2016

/cc @vsavkin , @tbosch as you may be touching these files as well

Pt 3 of migrating from package:guinness + package:unittest => package:test.

This PR migrates DirectiveProcessor & InlinerForTest unit tests.

Pt 3 of migrating from package:guinness + package:unittest => package:test.

This PR migrates DirectiveProcessor & InlinerForTest unit tests.
@kegluneq kegluneq added comp: dart-transformer action: review The PR is still awaiting reviews from at least one requested reviewer refactoring Issue that involves refactoring or code-cleanup labels Mar 8, 2016
@juliemr
Copy link
Copy Markdown
Member

juliemr commented Mar 8, 2016

LGTM once it looks good to Travis.

@juliemr juliemr added pr_state: LGTM and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Mar 8, 2016
@kegluneq
Copy link
Copy Markdown
Author

kegluneq commented Mar 8, 2016

@juliemr Thanks for the quick reviews

@kegluneq kegluneq added the action: merge The PR is ready for merge by the caretaker label Mar 8, 2016
@kegluneq kegluneq assigned vikerman and unassigned juliemr Mar 8, 2016
@kegluneq
Copy link
Copy Markdown
Author

kegluneq commented Mar 8, 2016

@vikerman , fyi this PR touches only test files -- there is no behavior change whatsoever.

@mary-poppins
Copy link
Copy Markdown

Merging PR #7475 on behalf of @vikerman to branch presubmit-vikerman-pr-7475.

@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 8, 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 refactoring Issue that involves refactoring or code-cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

X Tutup