项目中使用 rc-util/es/warning 后，打包后的生产代码为什么会包含警告代码？
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript, typescript, vite
- Área
- build-system, frontend
Línea de trabajo
Comienza con la reproducción de StackBlitz y el punto de entrada rc-util/es/warning; después, inspecciona las líneas 7356-7467 del dist/assets/index.57005310.js generado. Compara el bundle de producción con la implementación de warning mostrada en el issue; se considera terminado cuando el código de warning se elimina de la salida de producción sin romper el comportamiento en desarrollo.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
问题如题所述
复现案例
https://stackblitz.com/edit/vitejs-vite-zdg5hc?file=src%2FApp.tsx
请看案例中 dist/assets/index.57005310.js 文件中 7356-7467 代码行
期待行为：
正常来说生产代码不应该包含警告（console.log/console.warn/console.error）等不影响项目执行的代码
造成问题的原因:
// 源码实现如下(只是列出部分代码)
export function warning(valid, message) {
if (process.env.NODE_ENV !== 'production' && !valid && console !== undefined) {
console.error(message)
}
}
// 相比以上实现，下列实现则会在打包生产代码时移除相关代码
function noop() {}
export let warning = noop
if (process.env.NODE_ENV !== 'production') {
warning = function () {
if (!valid && console !== undefined) {
console.error(message)
}
}
}
另外，console !== undefined 可以移除，客户端和服务端不是都有 console 对象么？
- Lenguaje dominante
- TypeScript
- Estrellas
- 670
- Forks
- 206
- Merge medio
- 14 d 12 h
- PR fusionados (30 d)
- 3
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 react-component/util
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
react-component/util#816 ·
-
injectCSS method not attaching csp nonce passed to it. it is used in antd components internallyAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
react-component/util#671 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
react-component/util#738 · 1 comentario · 1 reacción ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
react-component/util#710 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
react-component/util#647 · 5 comentarios ·
Todos los issues de react-component/util
Issues similares
-
Add: PRO TV Chisinau SDAbiertocheck:failed streams:add
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
iptv-org/iptv#53974 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
interledger/rafiki#3986 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Doist/todoist-cli#576 ·
Los mantenedores suelen responder en 1 día
-
Suggestion: document (or optionally add) a cheaper-model config for find-skills on Claude CodeAbiertofeature
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
vercel-labs/skills#2370 ·
Los mantenedores suelen responder en 1 día
-
🐛 Bug supabase/cli
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día