Skip to content

Conversation

@pavoljuhas
Copy link
Collaborator

  • Delete the cirq-rigetti vendor module
  • Ignore deleted files in check/pylint-changed-files
  • Remove redundant pytest marker rigetti_integration
  • Remove cirq_rigetti references from typecheck configuration
  • Remove references to non-existing files from Python sources
  • Remove link to cirq-rigetti from the Cirq start notebook
  • Remove cirq-rigetti from Hardware and Reference sections of Cirq webpage
  • Delete cirq-rigetti web documentation and getting_started notebook

Resolves #7297

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions github-actions bot added the size: XL lines changed >1000 label Jul 15, 2025
@codecov
Copy link

codecov bot commented Jul 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.68%. Comparing base (89ebad8) to head (1117a84).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7488      +/-   ##
==========================================
- Coverage   98.68%   98.68%   -0.01%     
==========================================
  Files        1092     1091       -1     
  Lines       96855    96851       -4     
==========================================
- Hits        95583    95574       -9     
- Misses       1272     1277       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pavoljuhas pavoljuhas marked this pull request as ready for review July 15, 2025 02:10
@pavoljuhas pavoljuhas requested review from a team and vtomole as code owners July 15, 2025 02:10
@pavoljuhas pavoljuhas requested a review from mpharrigan July 15, 2025 02:10
@pavoljuhas
Copy link
Collaborator Author

NB: The leftover references to cirq_rigetti in .github/CODEOWNERS and
dev_tools/requirements/dev-np2.env.txt will be removed in separate PRs.

@pavoljuhas pavoljuhas requested review from dstrain115 and mhucka July 15, 2025 02:13
Copy link
Contributor

@mhucka mhucka left a comment

Choose a reason for hiding this comment

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

LGTM. There were a lot of bits involved!

@pavoljuhas pavoljuhas added this pull request to the merge queue Jul 15, 2025
github-merge-queue bot pushed a commit that referenced this pull request Jul 15, 2025
- Delete the `cirq-rigetti` vendor module
- Ignore deleted files in check/pylint-changed-files
- Remove redundant pytest marker `rigetti_integration`
- Remove `cirq_rigetti` references from typecheck configuration
- Remove references to non-existing files from Python sources
- Remove link to `cirq-rigetti` from the Cirq start notebook
- Remove `cirq-rigetti` from Hardware and Reference sections of Cirq
webpage
- Delete `cirq-rigetti` web documentation and `getting_started` notebook

Resolves #7297
Merged via the queue into quantumlib:main with commit 6d791fa Jul 15, 2025
35 checks passed
@pavoljuhas pavoljuhas deleted the delete-cirq-rigetti branch July 15, 2025 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: XL lines changed >1000

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove cirq-rigetti module

2 participants