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

Adding continuous integration. #3

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

christian-oreilly
Copy link

@christian-oreilly christian-oreilly commented Feb 6, 2025

fix #2

@christian-oreilly christian-oreilly linked an issue Feb 6, 2025 that may be closed by this pull request
6 tasks
@christian-oreilly
Copy link
Author

@Sabah98 Although the goal of the PR is to add continuous integration, I also addressed some packaging issues. For example, the most common structure is to have the source code nested in a folder of the name of the package (not "src"). Also, all modules required a __init__.py, even if this file is left empty.

@christian-oreilly
Copy link
Author

The tests on this PR won't pass but I will by-pass them when ready to merge because I don't want to spell check this before you merge your dev branch.

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.

Add continuous integration
1 participant