Skip to content

Conversation

@felps-dev
Copy link
Collaborator

Hoje se você der um SAVE com formatting ativado no VSCODE, muitos arquivos vão sofrer várias alterações, trabalhando com LLM's como Cursor eles automaticamente vão formatando todos os arquivos causando muitas alterações que impossibilitam acompanhar o que está sendo alterado.
Então eu rodei um ruff format . e formatei todos os arquivos para evitar isso, e também inclui no CI para verificar se a formatação está sendo aplicada caso alguém tente contribuir sem ter o ruff instalado.

  • Created requirements-dev.txt to include pytest and ruff for development.
  • Updated requirements.txt to reference the new requirements-dev.txt for development dependencies.
  • Modified CI configuration to install development dependencies and check formatting with ruff.
  • Cleaned up some docstrings and improved code formatting in various files for consistency.

…no CI

- Created requirements-dev.txt to include pytest and ruff for development.
- Updated requirements.txt to reference the new requirements-dev.txt for development dependencies.
- Modified CI configuration to install development dependencies and check formatting with ruff.
- Cleaned up some docstrings and improved code formatting in various files for consistency.
@leotada leotada requested a review from Copilot July 3, 2025 21:15
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR applies automated formatting across the repository using Ruff, centralizes development dependencies, and enforces formatting checks in CI.

  • Adds requirements-dev.txt for pytest and Ruff, updates requirements.txt, and modifies CI to install dev dependencies and run ruff check.
  • Reformats all test files for consistent quoting, trailing commas, and argument wrapping.
  • Cleans up minor docstrings and import lists for consistency.

Reviewed Changes

Copilot reviewed 60 out of 69 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/test_utils.py Reformatted imports, quotes, blank lines, and list literals
tests/test_nfse_serializacao_ginfes.py Standardized double quotes and regex string formatting
tests/test_nfse_serializacao_betha.py Consistent quoting in Betha serialization tests
tests/test_nfse_serializacao.py Reformatted arguments, quotes, and trailing commas
tests/test_nfe_serializacao_ii_di_ipi.py Wrapped long calls, standardized quotes and whitespace
tests/test_nfe_serializacao_geral.py Consolidated get_xsd calls into single-line wrapping
tests/test_nfe_serializacao_cst_*.py Applied uniform quoting, wrapped XPath expressions
Comments suppressed due to low confidence (1)

tests/test_utils.py:350

  • [nitpick] Consider extracting this long literal of UF codes into a module-level constant or fixture to improve readability and reduce inline verbosity in the test.
        lista_de_codigos_uf = [

@felps-dev felps-dev requested review from juniortada, leogregianin and leotada and removed request for juniortada, leogregianin and leotada July 15, 2025 22:27
@felps-dev felps-dev merged commit 98a92e1 into TadaSoftware:main Jul 16, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants