issue #8: Adding new workflow

[SamuelPelletierEvraire]

Tasks

• Research and choose a suitable workflow model (e.g., Gitflow, GitHub Flow).
• Define the branching strategy based on the selected workflow model.
• Set up branch protection rules to enforce the chosen workflow.
• Configure automated testing and code review processes within the workflow.

Acceptance Criteria

• The chosen workflow model is documented and clearly communicated to all team members.
• Branching strategy is established according to the selected workflow model.
• Branch protection rules are applied to enforce the workflow guidelines.
• Automated testing and code review processes are integrated into the workflow.

[rngadam]

fixed:

[rngadam]

@SamuelPelletierEvraire

The list of tasks has " Document the workflow guidelines and share them with the team."

• where's this documentation?

[SamuelPelletierEvraire]

For now i'm trying to fix all the issue that i have on the code The documentation was a mistake it is the same workflow as all the project of the ai lab

[rngadam]

For now i'm trying to fix all the issue that i have on the code and after i'm gonna make the documentation

so why is there a checkmark on the task?

[rngadam]

For now i'm trying to fix all the issue that i have on the code and after i'm gonna make the documentation

this is not the right approach ; you have a PR here that is about "adding a new workflow" so you should ensure that it is reviewed and merged. if you want to keep making changes that are unrelated to "adding a new workflow" you should branch off this one and work on that separetely so reviewers don't have to chase a constantly changing PR.

[SamuelPelletierEvraire]

@rngadam do you know why lint-test always fail I don't find why.

[SamuelPelletierEvraire]

For now i'm trying to fix all the issue that i have on the code and after i'm gonna make the documentation

this is not the right approach ; you have a PR here that is about "adding a new workflow" so you should ensure that it is reviewed and merged. if you want to keep making changes that are unrelated to "adding a new workflow" you should branch off this one and work on that separetely so reviewers don't have to chase a constantly changing PR.

I can merge because of those error so i need to fix them!

[rngadam]

it's trying to import tests but the folder is Tests with uppercase T.

this is why we have a naming convention to have all names lowercases.

the tests themselves don't follow the file naming convention.

pycache and compiled versions of the python scripts have also been erreneously checked in.

[rngadam]

description incorrect

[rngadam]

I don't think this should have been merged. it was far from done. it should have been rebased on main at least as the pyc and pycache are now checked in the history and the commit history looks really noisy: