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

deploy documentation in qiskit.org/ecosystem #937

Merged

Conversation

1ucian0
Copy link
Contributor

@1ucian0 1ucian0 commented May 5, 2023

Similar to Qiskit/qiskit-aer#1748, see Qiskit/qiskit.org#3038 for more context

Summary

The Qiskit Ecosystem lives in qiskit.org/ecosystem and new place for qiskit related projects that are not technically Qiskit. This PR adds that destination for documentation deployment.

Details and comments

For now, a copy of the same docs in ecosystem would allow to have both q.o/documentation and q.o/ecosystem for testing that everything is fine. Once that's done, the CDN will redirects to the new location and the deploy to q.o/documentation can be removed.

Copy link
Collaborator

@priti-ashvin-shah-ibm priti-ashvin-shah-ibm left a comment

Choose a reason for hiding this comment

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

Since they are all updates to comments, I can approve this.

LGTM>

@priti-ashvin-shah-ibm priti-ashvin-shah-ibm merged commit bbaaf01 into qiskit-community:main May 8, 2023
8 checks passed
@Eric-Arellano
Copy link
Contributor

Since they are all updates to comments, I can approve this.

To clarify, this new line is actual code:

rclone sync --progress ./docs/_build/html IBMCOS:qiskit-org-web-resources/ecosystem/metal

But Luciano has made this change successfully in ~10+ repos and this PR looks good to me. So this was safe to merge.

1ucian0 added a commit that referenced this pull request Sep 18, 2023
Follow up on #937

The documentation is correctly deployed in https://qiskit.org/ecosystem/metal/ . So, no need to upload to /documentation anymore.
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.

None yet

3 participants