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

Remove non-API docs and tutorials (Cherry-pick of #11352) #11398

Merged
merged 4 commits into from
Dec 14, 2023

Conversation

Eric-Arellano
Copy link
Collaborator

See #11352.

* Remove non-API docs and tutorials

* Remove sphinx-design

* Fix bad reference

* Fix bad deploy step

* Remove translatable strings script
@@ -18,9 +18,6 @@

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

FYI this file had merge conflicts. We had changed config that is now deleted.

@@ -1,286 +0,0 @@
qiskit.algorithms.AlgorithmError algorithms
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

FYI this file had merge conflicts. We had removed entries in 0.46, but now the whole file is deleted.

@@ -73,12 +73,10 @@ setenv =
{[testenv]setenv}
Copy link
Collaborator Author

@Eric-Arellano Eric-Arellano Dec 11, 2023

Choose a reason for hiding this comment

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

This had merge conflicts. I messed it up in the first and second commits, but I think it's now right.

@Eric-Arellano Eric-Arellano changed the title [wip] rRemove non-API docs and tutorials (Cherry-pick of #11352) Remove non-API docs and tutorials (Cherry-pick of #11352) Dec 12, 2023
@Eric-Arellano Eric-Arellano marked this pull request as ready for review December 12, 2023 13:59
@Eric-Arellano Eric-Arellano requested a review from a team as a code owner December 12, 2023 13:59
@qiskit-bot
Copy link
Collaborator

One or more of the the following people are requested to review this:

Copy link
Contributor

@ElePT ElePT left a comment

Choose a reason for hiding this comment

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

I have crossed checked the changes with #11352 and it looks like everything is in place. The only difference is a change in a file that doesn't exist in this branch (releasenotes/notes/platform-support-f7f693aaf5dec044.yaml). If the docs build worked in main I assume it will be fine here too.

@ElePT ElePT added this pull request to the merge queue Dec 14, 2023
Merged via the queue into Qiskit:stable/0.46 with commit 7a4a944 Dec 14, 2023
12 checks passed
@Eric-Arellano Eric-Arellano deleted the cp-rm-docs-46 branch December 14, 2023 15:40
Eric-Arellano added a commit to Eric-Arellano/qiskit-terra that referenced this pull request Dec 14, 2023
…it#11398)

* Remove non-API docs and tutorials (Qiskit#11352)

* Remove non-API docs and tutorials

* Remove sphinx-design

* Fix bad reference

* Fix bad deploy step

* Remove translatable strings script

* Fix bad merge conflict in tox.ini

* Delete contributing guides
github-merge-queue bot pushed a commit that referenced this pull request Dec 18, 2023
…11420)

* Remove non-API docs and tutorials (#11352)

* Remove non-API docs and tutorials

* Remove sphinx-design

* Fix bad reference

* Fix bad deploy step

* Remove translatable strings script

* Fix bad merge conflict in tox.ini

* Delete contributing guides
@hunterkemeny hunterkemeny added the Changelog: None Do not include in changelog label Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: None Do not include in changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants