Closed rmadhwal closed 2 years ago
Please ensure your PR passes the CI.
Sorry, seems like we accidentally commited changes to a submodule, could you please help rerun the workflow
I approved the workflow 👍
Also, please clean up your PR history by squashing it to a single, or a few commits.
Sure, however we're still making small final touches and would presumably also have to make changes after your reviews, so we'll squash after all the changes have been reviewed. Also could someone help approve the workflow again ? :)
@rmadhwal sure!
Hmm I was hoping that the workflow wouldn't need explicit permission every time...
Dear FoC team, your code quality is not up to par and is failing our CI. Please fix the issues laid out in my comments in order for you to move on to the final grading phase of this course.
Dear FoC team, your code quality is not up to par and is failing our CI. Please fix the issues laid out in my comments in order for you to move on to the final grading phase of this course.
Thanks for the suggestions, we are currently working on implementing them, we'll have the fixes in tomorrow
We have implemented the suggested changes, and added some extra functionalities:
We are now ready for submissions and move to the final grading phase.
@InvictusRMC we have made the requested changes, could you confirm if the code quality is fine now ?
@InvictusRMC We added your suggestions.
This is a pull request for Team Contribute Intelligence's progress in Week 8 Our Main Contributions so far are:
https://user-images.githubusercontent.com/14342304/160671450-ac2deac8-3516-436b-aa39-dcb7be6fbd85.mp4
For a more detailed breakdown of our work please refer to our open issue