chore(files): align PDF viewer import order - #7846
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
|
| import { bindPreviewWheelZoom } from '@sim/emcn' | ||
| import { createLogger } from '@sim/logger' | ||
| import { pdfjs, Document as ReactPdfDocument, Page as ReactPdfPage } from 'react-pdf' |
There was a problem hiding this comment.
The reordered imports place the @sim/emcn UI component import before the external @sim/logger and react-pdf imports. This violates the repository directive that external libraries must come before UI components, so the required import order must be restored before merging.
| import { bindPreviewWheelZoom } from '@sim/emcn' | |
| import { createLogger } from '@sim/logger' | |
| import { pdfjs, Document as ReactPdfDocument, Page as ReactPdfPage } from 'react-pdf' | |
| import { createLogger } from '@sim/logger' | |
| import { pdfjs, Document as ReactPdfDocument, Page as ReactPdfPage } from 'react-pdf' | |
| import { bindPreviewWheelZoom } from '@sim/emcn' |
Context Used: CLAUDE.md (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Summary
Type of Change
Testing
Checklist