Repository navigation
Auditoría del CLI y versión 4.0.462 - #3
Merged
Merged
Conversation
- Se quitan los comentarios de src/ y tests/. Las razones de los cambios viven en este historial. - Mensajes, descripciones de ayuda, etiquetas de tablas y prompts pasan a inglés y se acortan. Comandos, opciones, códigos de salida, forma de --json, variables de entorno y almacenes de credenciales no cambian. - GetValue(x)! pasa a GetRequiredValue(x) cuando la opción es obligatoria o tiene default. Las opcionales quedan nullable y se manejan explícitamente. Se eliminan los ! de src/. - El catch del aviso de versión se limita a HttpRequestException y JsonException. Las pruebas de UpdateCheck cubren cada falla; se agregan dos pruebas de rutas de estado inválidas que fallan sin el cambio. - webhooks delete acepta solo y/yes en la confirmación. - La línea de entrega de listen no lleva flecha. - Las pruebas se actualizan al texto nuevo. Resultado: 404 de 404 en verde, 0 warnings.
- README con el orden: qué es, instalación, autenticación, uso, comandos, solución de problemas y licencia. Se quitan las secciones de desarrollo y de releases. - La verificación del SHA-256 es opcional y va en su propia sección. - Sin menciones de .NET ni de detalles de implementación. - Los ejemplos de salida se alinean con lo que imprime el CLI: marcadores de --help, líneas de listen, aviso de versión y errores de uso. Los textos que devuelve la API se citan tal cual. - Los secretos y trazas de ejemplo son placeholders. - Enlaces verificados; markdownlint sin errores. - Se agrega LICENSE con la Mozilla Public License 2.0.
- FiscalApi.Cli.csproj: Version 4.0.462. release.yml toma la versión del tag o de la entrada manual, valida semver y compara el --version del binario contra ella. - Directory.Build.props y el .csproj de pruebas pierden sus comentarios XML; sus propiedades no cambian. La documentación XML no está activada, así que no hay CS1591 que atender. - Los workflows y scripts de .github pierden sus comentarios; los SHA de las acciones siguen fijados. Se quitan los IDs internos y el texto de infraestructura de los comentarios. - .markdownlint-cli2.jsonc pierde su encabezado de comentarios.
- Se quitan los comentarios de .github/workflows y de los scripts. Los SHA de las acciones siguen fijados; el comentario de versión de cada acción va con él. - Se eliminan los IDs internos y el texto de infraestructura de los comentarios. Los nombres de los pasos de CI (Native AOT, AlmaLinux 8) se conservan, porque son parte de la ejecución. - Los shebangs y el contenido de los scripts no cambian.
|
Important Review skippedToo many files! This PR contains 156 files, which is 56 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configuration
📒 Files selected for processing: 156
You can disable this status message by setting the
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Resumen
Auditoría de extremo a extremo del CLI: código, pruebas, workflows, README y
docs/. La versión pasa a 4.0.462, la versión actual del backend.Cambios visibles para el usuario
--json, las variables de entorno y el almacenamiento de credenciales no cambian.listensin flecha. Pasa dehora --> tipo [id] [código]ahora tipo [id] [código].yoyes.sysiya no confirman. El prompt muestra[y/N].HttpRequestException) y de JSON (JsonException) se silencian. Otras excepciones ya no se tragan en silencio.docs/describen instalación, autenticación, uso y referencia de comandos. La verificación del SHA-256 es opcional. La guía de webhooks se enlaza endocs.fiscalapi.com/webhooks-info.LICENSE.Cambios internos
src/,tests/,Directory.Build.props, los.csproj, los workflows de.github/y.markdownlint-cli2.jsonc.GetRequiredValue. Se quitan los!injustificados y los casts sin validar.ApiCommandContext, que lanza una excepción explícita si la API devuelve un cuerpo vacío.catchdel aviso de versión (ver arriba). Se agregan pruebas para la ruta del directorio de estado.publish-linux.sh.Verificación
dotnet build -c Release: 0 warnings, 0 errores.dotnet test: 404 de 404 en verde.fiscalapi --versionimprime4.0.462.--helpdelogin,logout,listen,trigger,webhooks list|create|deleteyevents list|retrievesale con código 0.markdownlint-cli2sobreREADME.mdydocs/*.md: 0 issues.Pendiente antes de publicar
login,trigger,listencontra una sesión viva ni los errores 400/401/403/409 que devuelve la API. Los ejemplos que dependen de la API están marcados como no verificados.release.ymlal publicar un tag.