-
Notifications
You must be signed in to change notification settings - Fork 2
Adding continuous integration. #3
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
base: main
Are you sure you want to change the base?
Conversation
|
@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 |
|
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. |
|
@dw42CSCE This branch is now rebased against the main so you can @Sabah98 Please mind that this branch rename the |
Co-authored-by: Christian O'Reilly <[email protected]>
fix #2