Skip to content

Conversation

@tbouffard
Copy link
Member

@tbouffard tbouffard commented Jan 13, 2022

The build often has errors and they are only detected after a developer pull the development branch on their local machine.
See #61 for instance.
Currently, we have errors due to some imports that are not respecting the case of the filenames. I guess this works on Windows but it cannot work on Linux.
To early catch build failures, I propose to add a GitHub workflow that runs a build command. This pull request also provide fixes for the imports.

Note that this is an initial version of the build automation. Feel free to comment if you see areas for improvements. But I would like we start we something quickly. We will be able to improve soon or later.

@tbouffard tbouffard marked this pull request as ready for review January 13, 2022 17:14
@tbouffard tbouffard requested review from junsikshim and mcyph January 13, 2022 17:15
@mcyph
Copy link
Collaborator

mcyph commented Jan 13, 2022

Looks good to me!

Regarding casing of files, I intended to make modules which have functions camelCase and classes InitialCaps but looks git didn't rename them - I think if we want to keep with this convention we might need to do 2 commits to rename to a different name before changing back to lowercase, but we can worry about this later.

@tbouffard tbouffard merged commit fb7819f into development Jan 14, 2022
@tbouffard tbouffard deleted the chore/fix_build_issues_and_introduce_gh_workflow branch January 14, 2022 05:17
@tbouffard tbouffard added the chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...) label Nov 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants