amazon-braket / autoqasm

AutoQASM is an experimental module offering a new quantum-imperative programming experience in Python for developing quantum programs.
Apache License 2.0
15 stars 9 forks source link

feature: support integer division #29

Closed abidart closed 5 months ago

abidart commented 5 months ago

Issue #, if available:

9 and builds on the work of @daredevil3435

Description of changes:

Testing done: tox -e unit-tests + added unit tests.

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

Tests

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (24e6fa1) to head (f44b48e). Report is 10 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #29 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 44 46 +2 Lines 1991 2043 +52 Branches 337 342 +5 ========================================= + Hits 1991 2043 +52 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

daredevil3435 commented 5 months ago

šŸ„³ Looks great to me. Thank you, @abidart and @daredevil3435!

Yes. My very first bug bounty!! Thanks you so much for this @rmshaffer šŸ˜

rmshaffer commented 5 months ago

@daredevil3435 and for your first bounty, this was not a simple one at all! Thanks for sticking with it šŸ‘

daredevil3435 commented 5 months ago

@daredevil3435 and for your first bounty, this was not a simple one at all! Thanks for sticking with it šŸ‘

But you made it look really simple!