Add missing controller guidelines
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 67/100
- Tipo de issue
- Documentación
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- typescript
- Área
- documentation
Línea de trabajo
Encuentra las directrices existentes para controladores y paquetes en el repositorio; el issue no indica sus archivos. Lee las directrices circundantes y, después, incorpora donde corresponda los puntos indicados sobre el estado del controlador, la configuración de pruebas, el mocking de messenger y las notas sobre versiones de paquetes. Se considerará terminado cuando las directrices relevantes cubran estos puntos sin duplicar ni contradecir las directrices existentes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
We have some missing controller guidelines:
- State should be coherent at all times. If properties are intended to be used in conjunction with each other they should be consistent, and we should have runtime validation to ensure that this is always the case.
- Use
withControllerorsetupControllerpattern to set up controllers in tests. - Don't mock the messenger in tests.
Although the following is a package guideline, we could consider repeating it in the controller guidelines because that is probably more likely the use case:
- New controller packages should always start at 0.0.0 (otherwise they run the risk of being accidentally published).
- Lenguaje dominante
- TypeScript
- Estrellas
- 414
- Forks
- 309
- Merge medio
- 22 h 25 min
- PR fusionados (30 d)
- 367
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin guía de contribución
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 MetaMask/core
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
platform-api-docs: `scan` strategy misses ESM-only packages, which ship `.d.ts` rather than `.d.cts`Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
[utils] Replace the `uuid` dependency with `crypto.randomUUID()`Posiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
MetaMask/core#10210 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
eth-json-rpc-middleware: `TransactionParamsStruct` rejects numeric `chainId`, breaking dapp transactionsPosiblemente ocupada @truehazker la tomó hace 46 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
MetaMask/core#9964 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de MetaMask/core
Issues similares
-
Table: Space fires onActivate in single-selection mode — the reference doc and the JSDoc disagreeAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
sidorares/react-x11-components#764 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
backnotprop/plannotator#1840 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
JoviDeCroock/pracht#432 ·
Los mantenedores suelen responder en 1 día
-
Add: CNN en Espanol SDAbiertoapproved check:passed streams:add
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Hardware attribute name "app Connection Support" has inconsistent casingPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
walletbeat/walletbeat#1628 ·
Los mantenedores suelen responder en 1 día