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

Refacto: modernizing and optimizing source codebase #256

Merged
merged 60 commits into from
May 24, 2024

Conversation

Guts
Copy link
Owner

@Guts Guts commented May 20, 2024

  • remove deprecated code raising warnings
  • replace dict by models based on dataclasses
  • factorize Georeaders using a based-class POO approach

@github-actions github-actions bot added the CLI Command Line Interface label May 20, 2024
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Comment on lines 1027 to 1030
f"postgresql://{metadataset.database_connection.user_name}"
f"@{metadataset.database_connection.host}"
f":{metadataset.database_connection.port}"
f"/{metadataset.database_connection.database_name}"

Check failure

Code scanning / SonarCloud

A secure password should be used when connecting to a database

<!--SONAR_ISSUE_KEY:AY-XNKsRhBGlqlyJ0GG--->Add password protection to this database. <p>See more on <a href="https://sonarcloud.io/project/issues?id=Guts_DicoGIS&issues=AY-XNKsRhBGlqlyJ0GG-&open=AY-XNKsRhBGlqlyJ0GG-&pullRequest=256">SonarCloud</a></p>
Copy link

codecov bot commented May 21, 2024

Codecov Report

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

Project coverage is 26.42%. Comparing base (ad9e9ff) to head (db6ecd3).
Report is 2 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #256       +/-   ##
===========================================
+ Coverage   15.68%   26.42%   +10.74%     
===========================================
  Files          34       37        +3     
  Lines        2856     2539      -317     
  Branches      296      321       +25     
===========================================
+ Hits          448      671      +223     
+ Misses       2376     1823      -553     
- Partials       32       45       +13     
Flag Coverage Δ
unittests 26.42% <41.53%> (+10.74%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
dicogis/__about__.py 100.00% <100.00%> (ø)
dicogis/constants.py 96.87% <100.00%> (+3.32%) ⬆️
dicogis/models/feature_attributes.py 100.00% <100.00%> (ø)
dicogis/utils/journalizer.py 94.23% <100.00%> (+0.61%) ⬆️
dicogis/utils/slugger.py 100.00% <100.00%> (ø)
dicogis/utils/texts.py 87.50% <94.11%> (+7.50%) ⬆️
dicogis/cli/main.py 0.00% <0.00%> (ø)
dicogis/georeaders/read_dxf.py 0.00% <0.00%> (ø)
dicogis/models/database_connection.py 82.75% <82.75%> (ø)
dicogis/georeaders/read_vector_flat_geodatabase.py 0.00% <0.00%> (ø)
... and 14 more

@Guts Guts added this to the 4.0.0-beta5 milestone May 21, 2024
@Guts Guts changed the title Refacto: dataclasses' models instead of dict Refacto: modernizing and optimizing source codebase May 22, 2024
@github-actions github-actions bot added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file labels May 23, 2024
@Guts Guts force-pushed the refacto/dataclass-models-instead-of-dict branch from 64a421b to db6ecd3 Compare May 24, 2024 09:32
Copy link

sonarcloud bot commented May 24, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@Guts Guts merged commit db418a6 into master May 24, 2024
8 of 9 checks passed
@Guts Guts deleted the refacto/dataclass-models-instead-of-dict branch May 24, 2024 09:40
@Guts Guts added enhancement New feature or request and removed documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file labels May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLI Command Line Interface enhancement New feature or request quality tooling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant