yarn lint
https://dagster.phacility.com/P74
Details
Details
Diff Detail
Diff Detail
- Repository
- R1 dagster
- Branch
- yuhan/doc-issues
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Comment Actions
This looks great, thanks for doing this! Can you explain the regex change - just want to make sure that won't break any links in a weird way
| docs/next/src/__tests__/mdxInternalLinks.test.ts | ||
|---|---|---|
| 76–77 | Delete? | |
| docs/next/src/components/AnchorHeading.tsx | ||
| 33 | Can you explain the change here? | |
| docs/next/src/components/Search.tsx | ||
| 32 | Delete | |
| 74 | This works 👍 | |
| docs/next/src/components/Transition.js | ||
| 2 | I would double check if you're supposed to import React I think you have to import it in components, but not pages, so this looks right. | |