Skip to content

Fix CI: duplicate imports, same-slug tests and unclassified autolink routes - #246

Closed
danielehrhardt wants to merge 2 commits into
mainfrom
fix/ci-duplicate-imports
Closed

danielehrhardt wants to merge 2 commits into
mainfrom
fix/ci-duplicate-imports

Conversation

@danielehrhardt

@danielehrhardt danielehrhardt commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

Main has been red since #241, #243 and #245 were merged one after the other (run 40173455998).

Checked locally: core/shared/desktop typecheck clean, remote-snapshot + cloud-scope tests 50/50 pass.

The merge left duplicate group/replica imports in remote-snapshot.test.ts, and
#241's two same-name tests expected the secondary's project to be skipped. With
#243 the local project steps aside to a free slug instead and both are kept, so
the tests now expect that.
Reading the suggestions is allowed like GET /api/totp; linking them needs secret
access like linking a 2FA code from a missing login.
@danielehrhardt danielehrhardt changed the title Fix CI: duplicate imports and same-slug tests after #241 + #243 Fix CI: duplicate imports, same-slug tests and unclassified autolink routes Oct 25, 2026
@danielehrhardt

Copy link
Copy Markdown
Contributor Author

Superseded by #247 and #248, which landed the same fixes on main.

@danielehrhardt
danielehrhardt deleted the fix/ci-duplicate-imports branch October 25, 2026 08:13
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.

1 participant