Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🔧 Melhorando arquitetura e corrigindo bugs #9

Merged
merged 3 commits into from
May 31, 2024
Merged

Conversation

AyslanBatista
Copy link
Owner

@AyslanBatista AyslanBatista commented May 31, 2024

  • Melhorado a arquitetura do projeto, unificando cryptcode.py no core.py
  • Removendo funções redundantes
  • Novo arquivo utils.py para funções auxiliares
  • [BUG] Corrigindo o tratamente de erros do arquivo core.py da função process_file
  • [BUG] Corrigindo erro no CLI para caminhos absolutos em --keyfile e --file
  • [BUG] Corrigido palavras errados no template.py
  • [Test] - Cobertura de test para as funções do arquivo utils.py
  • [Codacy] Corrigindo as 3 issues de possível falha de segurança no codigo

- Melhorado a arquitetura do projeto, unificando cryptcode.py no core.py
- Removendo funções redundantes
- Novo arquivo utils.py para funções auxiliares
- [BUG] Corrigindo o tratamente de erros do arquivo core.py da função process_file
- [BUG] Corrigindo erro no CLI para caminhos absolutos em --keyfile e --file
- [BUG] Corrigido palavras errados no template.py
@AyslanBatista AyslanBatista changed the title 🔧 Melhorando qualidade do codigo 🔧 Melhorando arquitetura e corrigindo bugs May 31, 2024
@AyslanBatista AyslanBatista force-pushed the melhorias branch 2 times, most recently from 1e23b76 to 5fd82be Compare May 31, 2024 23:22
@AyslanBatista AyslanBatista added bug Something isn't working enhancement New feature or request python Pull requests that update Python code labels May 31, 2024
@AyslanBatista AyslanBatista self-assigned this May 31, 2024
- [Test] - Cobertura de test para as funções do arquivo utils.py
- [Codacy] Corrigindo as 3 issues de possível falha de segurança no codigo
Copy link

Test Result (Python 3.11)

9 tests  +5   9 ✅ +5   0s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit a514bcb. ± Comparison against base commit a69725e.

Copy link

Test Result (Python 3.12)

9 tests  +5   9 ✅ +5   0s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit a514bcb. ± Comparison against base commit a69725e.

Copy link

Test Result (Python 3.10)

9 tests  +5   9 ✅ +5   0s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit a514bcb. ± Comparison against base commit a69725e.

Copy link

codecov bot commented May 31, 2024

Codecov Report

Attention: Patch coverage is 40.00000% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 18.04%. Comparing base (a69725e) to head (a514bcb).

Files Patch % Lines
encryptdef/cli.py 0.00% 4 Missing ⚠️
encryptdef/interactive_interface.py 0.00% 4 Missing ⚠️
encryptdef/__main__.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
+ Coverage   13.67%   18.04%   +4.36%     
==========================================
  Files           8        8              
  Lines         329      338       +9     
==========================================
+ Hits           45       61      +16     
+ Misses        284      277       -7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@AyslanBatista AyslanBatista merged commit d90d0dc into main May 31, 2024
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant