Skip to content

Comments

Dev#10

Open
hamdi3 wants to merge 25 commits intomainfrom
dev
Open

Dev#10
hamdi3 wants to merge 25 commits intomainfrom
dev

Conversation

@hamdi3
Copy link
Owner

@hamdi3 hamdi3 commented Dec 22, 2024

No description provided.

hamdi3 and others added 25 commits September 21, 2024 11:34
Fixed imports for PKCS1 to use it a signature instead of a cipher
Integrated Flask into it and implemented the routes
Added routes tests
Added Flask to req.
Added docker support
Added simple transaction validators
Updated the tests for transaction validators
Updated the route tests to cover the cases of walets not being found
Updated tests to improve the coverage
Removed the transaction class and added the blockchain class
Updated the routes based on the new blockchain class & added more routes
Updated the tests
Removed the version from the docker-compose.yaml file since it's obsolete
Updated the requirements.txt file
Code fixes
- Added OpenAI(Swagger) to document the api.
- Updated the routes to use JSON instead of forms (for Consistency, flexibility, scalability).
Updated the requirements.txt
- Updated the routes tests to accept JSON instead of forms
- Increased coverage
Fixed some linters warnings
Added a logging configuration
Added some loggers to the code
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 94.91525% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/blockchain.py 90.47% 2 Missing ⚠️
tests/test_blockchain.py 90.00% 1 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines Coverage Δ
src/logging_config.py 100.00% <100.00%> (ø)
tests/test_routes.py 99.63% <100.00%> (ø)
tests/test_blockchain.py 99.05% <90.00%> (ø)
src/blockchain.py 94.32% <90.47%> (-0.68%) ⬇️

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