Comments that apply to a whole file or the whole diff, not a line
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- typescript
- Área
- tooling
Línea de trabajo
Start by locating the existing line-comment UI and the import/export paths that serialize anchors. Review the /diffx-finish-review skill documentation as part of the scope. Done means file- and diff-level comments work without changing line comments, older exports still import, and duplicate detection includes scope.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
Every comment is anchored to a line (or line range, #33). Observations about a file as a whole ("this module should be split") or the change as a whole ("missing migration") have no anchor, so reviewers attach them to an arbitrary line.
Proposal
- File-level comment: a "Comment" button in each file header, anchored to the file path only.
- Diff-level comment: a "Comment on diff" button above the first file, anchored to nothing.
- "Copy comments" emits
<comment scope="file">inside the<file>block and<comment scope="diff">outside all<file>blocks. Import parses both and de-duplicates on scope as well as anchor. - Line comments are unchanged on disk and in export (
scopeabsent meansline).
Compatibility
Older exports import unchanged. The /diffx-finish-review skill documentation requires an update for the two new scopes. A PR follows.
- Lenguaje dominante
- TypeScript
- Estrellas
- 206
- Forks
- 37
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de wong2/diffx
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 38/100
Todos los issues de wong2/diffx
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
lichess-org/api#678 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
PostHog/posthog.com#20628 ·
Los mantenedores suelen responder en 1 día
-
bug status:Needs Triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
jupyterlab/jupyterlab#19964 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
agentscope-ai/QwenPaw#8064 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
area: notebooks-jupyter bug theme: new notebook frontend
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
posit-dev/positron#16347 · 1 comentario ·
Los mantenedores suelen responder en 1 día