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

Womanium Assignment #284

Closed
wants to merge 9 commits into from
Closed

Womanium Assignment #284

wants to merge 9 commits into from

Conversation

NoahNzeki
Copy link
Contributor

@NoahNzeki NoahNzeki commented Jul 25, 2024

PR Description

Some notes

  • Please make sure that you placed the files in an appropriate folder

  • And that the files have indicative names.

  • Please note that Classiq runs automatic code linting, which may minorly alter some files.

    • If you're familiar with pre-commit, you may run pre-commit install, and then at each commit, your files will be altered in a similar way

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Contributor Author

@NoahNzeki NoahNzeki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed Submission

@Nadav138 Nadav138 added the Womanium Womanium 2024 Summer Program label Aug 4, 2024
@Nadav138
Copy link
Contributor

Nadav138 commented Aug 5, 2024

Good job!
Score: 8/10.
I closed the PR as it contains two not relevant files (from QClass). If you wish, you can reopen a PR.

@Nadav138 Nadav138 closed this Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Womanium Womanium 2024 Summer Program
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants