Skip to content
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

refactor!: remove SlowComponentLoader and ReflectiveInjector. #26

Merged
merged 2 commits into from
Sep 14, 2022

Conversation

ykmnkmi
Copy link

@ykmnkmi ykmnkmi commented Sep 8, 2022

BREAKING CHANGE: remove SlowComponentLoader, ReflectiveInjector classes with dependent runAppLegacy, runAppLegacyAsync and bootstrapStatic functions.

@GZGavinZhao
Copy link

I'm trying to enforce conventional commit now, so if you want to squash & merge this PR as one big commit, please rename the PR title (which will become the commit title) when you submit this as ready for review. Otherwise, when submitting, please rebase/edit/squash/force push or whatever you like to ensure individual commits follow conventional commit guidelines.

@ykmnkmi ykmnkmi changed the title Remove SlowComponentLoader and ReflectiveInjector. refactor!: remove SlowComponentLoader and ReflectiveInjector. Sep 9, 2022
@ykmnkmi ykmnkmi marked this pull request as ready for review September 9, 2022 05:40
@GZGavinZhao GZGavinZhao self-assigned this Sep 10, 2022
@GZGavinZhao GZGavinZhao self-requested a review September 10, 2022 00:15
@@ -4,6 +4,19 @@ import 'package:test/test.dart';

import 'directive_test.template.dart' as ng;

Object missingServicFactory(Object willNotBeCalled) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missingServiceFactory?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated.

Copy link

@GZGavinZhao GZGavinZhao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thank you!

@GZGavinZhao
Copy link

Just one small thing, in the function I commented on, did you mean to type missingServiceFactory instead of missingServicFactory?

@ykmnkmi
Copy link
Author

ykmnkmi commented Sep 14, 2022

Yes.

@GZGavinZhao GZGavinZhao merged commit 55682cb into angulardart-community:master Sep 14, 2022
GZGavinZhao pushed a commit that referenced this pull request Sep 14, 2022
@ykmnkmi ykmnkmi deleted the clean branch February 22, 2023 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants