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(core): better error message for undecorated-classes-with-di migration #33315
refactor(core): better error message for undecorated-classes-with-di migration #33315
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@devversion Can you rebase? |
…migration Currently if one of the project targets could not be analyzed due to AOT compiler program failures, we gracefully proceed with the migration. This is expected, but we should not print a message at the end of the migration that the migration was _successful_. The migration was only done partially, hence it's potentially incomplete and we should make it clear that once the failures are resolved, the migration should be re-run.
338f571
to
18e5bad
Compare
…migration (angular#33315) Currently if one of the project targets could not be analyzed due to AOT compiler program failures, we gracefully proceed with the migration. This is expected, but we should not print a message at the end of the migration that the migration was _successful_. The migration was only done partially, hence it's potentially incomplete and we should make it clear that once the failures are resolved, the migration should be re-run. PR Close angular#33315
…migration (angular#33315) Currently if one of the project targets could not be analyzed due to AOT compiler program failures, we gracefully proceed with the migration. This is expected, but we should not print a message at the end of the migration that the migration was _successful_. The migration was only done partially, hence it's potentially incomplete and we should make it clear that once the failures are resolved, the migration should be re-run. PR Close angular#33315
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Currently if one of the project targets could not be analyzed
due to AOT compiler program failures, we gracefully proceed
with the migration. This is expected, but we should not
print a message at the end of the migration that the migration
was successful. The migration was only done partially, hence
the migration is potentially incomplete and we should make it clear that once
the failures are resolved, the migration should be re-run.