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

core: shift each component to its folder #85851

Merged
merged 1 commit into from
Feb 25, 2025
Merged

Conversation

JonasBa
Copy link
Member

@JonasBa JonasBa commented Feb 25, 2025

Remove alertlink export backcompat after getsentry is merged

@JonasBa JonasBa added the Scope: Frontend Automatically applied to PRs that change frontend components label Feb 25, 2025
Base automatically changed from jb/core/imports to master February 25, 2025 16:01
@JonasBa JonasBa requested review from a team as code owners February 25, 2025 16:01
@JonasBa JonasBa force-pushed the jb/core/imports-alertlink branch from 1631a44 to 51963c3 Compare February 25, 2025 16:50
Copy link

codecov bot commented Feb 25, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
8365 1 8364 3
View the top 1 failed test(s) by shortest run time
AccountSecurity cannot enroll without verified email
Stack Traces | 1.17s run time
Error: expect(jest.fn()).toHaveBeenCalled()

Expected number of calls: >= 1
Received number of calls:    0

Ignored nodes: comments, script, style
...
    at .../account/accountSecurity/index.spec.tsx:213:78
    at runWithExpensiveErrorDiagnosticsDisabled (.../sentry/sentry/node_modules/@.../dom/dist/config.js:47:12)
    at checkCallback (.../sentry/sentry/node_modules/@.../dom/dist/wait-for.js:124:77)
    at checkRealTimersCallback (.../sentry/sentry/node_modules/@.../dom/dist/wait-for.js:118:16)
    at Timeout.task [as _onTimeout] (.../sentry/sentry/node_modules/jsdom/lib/jsdom/browser/Window.js:520:19)
    at listOnTimeout (node:internal/timers:594:17)
    at processTimers (node:internal/timers:529:7)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@JonasBa JonasBa force-pushed the jb/core/imports-alertlink branch from 51963c3 to 1bf541a Compare February 25, 2025 16:57
@JonasBa JonasBa merged commit 26b0863 into master Feb 25, 2025
34 checks passed
@JonasBa JonasBa deleted the jb/core/imports-alertlink branch February 25, 2025 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants