Contribution to the projects Software Engineering: implementation of CI using Docker #15
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.
Description
Me and my friend Eduardo Schuindt have made a contribution to CoCa-pytorch as a class project in our University. Our contribution implements CI in your project's pipeline, using Docker to build the application and deploy the library to PyPI.
Although our knowledge in your project's field is limited, we could contribute to your project's software engineering, enhancing its quality and performance. We used Sphinx to create an automated documentation, Poetry to build and publish the Python package to PyPI and Docker to create containers. Also, it was necessary to re-arrange some folder's structure to assure our features would work properly. Everything we did can be easily changed and enhanced as needed in the future.
There are still some enhancements to be done. Some adjusts may be needed to finish poetry publish (as it requires authentication of the publisher), implement tests (they should be working in our pipeline as soon as you create some tests in the tests folder) and, if you wish, there is already a Docker container to build a database.
Using Docker build processes simplifies the deployment procedures and creates consistency across different development environments, so we strongly recommend using it.
This pull request has all the necessary changes. Your main code file is now named main.py and is in the folder named src. There were also adjustments to readme, making it easy to follow the steps needed. The requirements were also corrected, as it used to have compatibility problems.
This PR has only 1 commit due to the professors requirements. The complete history may be found in this repository.
We would be grateful if you reviewed this PR and gave a feedback or approval.
All the best,
Ana Hoffmann and Eduardo Schuindt