Skip to content

improved_qhed#1381

Merged
TomerGoldfriend merged 1 commit intoClassiq:mainfrom
yuvalc-lab:improve_qhed
Dec 14, 2025
Merged

improved_qhed#1381
TomerGoldfriend merged 1 commit intoClassiq:mainfrom
yuvalc-lab:improve_qhed

Conversation

@yuvalc-lab
Copy link
Contributor

PR Description

Some notes

  • Please make sure that the notebook runs successfully with the latest Classiq version.

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

    • And that the file names are clear, descriptive, and match the notebook content.
      • Note that we require the file names of .ipynb and .qmod to be unique across this repository.
    • Plus, please make sure that all required files are included: .qmod, .synthesis_options.json, .metadata.json
    • And that images are embedded inside the notebook, not added as external files
  • If applicable, please include link to the paper on which the notebook is based, in the notebook itself.

  • Please use rebase on your branch (no merge commits)

  • Please link this PR to the relevant issue

  • Please make sure to run pre-commit when commiting changes

    • If you're using git in the terminal, make sure to install pre-commit via running pip install pre-commit followed by pre-commit install
    • Note that Classiq runs automatic code linting. Meaning that one of the tests verifies the output of pre-commit.
    • Also note that pre-commit may minorly alter some files. Make sure to git add the changes done by pre-commit

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@TomerGoldfriend TomerGoldfriend merged commit 273bf78 into Classiq:main Dec 14, 2025
3 checks passed
@github-actions
Copy link

🚀 Fantastic work @yuvalc-lab! Your very first PR to classiq-library has been merged! 🎉🥳

If you're feeling adventurous, why not dive into another issue and keep contributing? The Classiq community would love to see more from you! 🚀

Happy coding! 👩‍💻👨‍💻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants