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
fix(ember): align span operations to new operations #5883
Conversation
'ember.route.setupController | slow-loading-route', | ||
'ember.route.setupController | slow-loading-route.index', | ||
'ui.ember.route.setup_controller | slow-loading-route', | ||
'ui.ember.route.setup_controller | slow-loading-route.index', | ||
'ui.ember.runloop.routerTransitions | undefined', | ||
'ui.ember.component.render | component:link-to', | ||
'ui.ember.component.render | component:link-to', |
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.
Should I change ember.route.model
operation in this file?
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.
Yes we should change that to prepend ui
Hey @outsideris thanks for opening a PR! Could you split this up so that you create a PR for each package (one PR for ember, one PR for serverless, etc.)? I know that’s a little more work, but it’ll make it easier to review changes (plus it’ll be more contributions for you 😄) |
Sure, I will split them. |
I updated this PR for only |
Signed-off-by: Outsider <outsideris@gmail.com>
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.
Thanks for splitting everything up!
Ref: #5837
As the description, I changed span operations to new operations. Is it right?
My local development environment is not the same as CI environments, so I will fix some broken lint or tests after I can see CI results.