Skip to content

fix(ember): align span operations to new operations - #5883

Merged
AbhiPrasad merged 1 commit into
getsentry:masterfrom
outsideris:issue-5837
Oct 6, 2022
Merged

fix(ember): align span operations to new operations#5883
AbhiPrasad merged 1 commit into
getsentry:masterfrom
outsideris:issue-5837

Conversation

@outsideris

@outsiderisoutsideris commented Oct 4, 2022

Copy link
Copy Markdown
Contributor

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.

'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',

Copy link
Copy Markdown
ContributorAuthor

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?

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.

Yes we should change that to prepend ui

@AbhiPrasad

Copy link
Copy Markdown
Contributor

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 😄)

@outsideris

Copy link
Copy Markdown
ContributorAuthor

Sure, I will split them.

@outsiderisoutsideris changed the title fix(span): align span operations to new operationsfix(ember): align span operations to new operationsOct 5, 2022
@outsideris

Copy link
Copy Markdown
ContributorAuthor

I updated this PR for only packages/ember.
Because my local environment is weird for packages/ember, I can't urn test for packages/ember.
I'm waiting the CI results.

Signed-off-by: Outsider <outsideris@gmail.com>

@AbhiPrasadAbhiPrasad left a comment

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.

Thanks for splitting everything up!

@AbhiPrasad
AbhiPrasad merged commit 912f5f1 into getsentry:masterOct 6, 2022
@AbhiPrasadAbhiPrasad mentioned this pull request Oct 6, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@outsideris@AbhiPrasad