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

Edits to contributing #5

Merged
merged 1 commit into from
Dec 26, 2018
Merged

Edits to contributing #5

merged 1 commit into from
Dec 26, 2018

Conversation

jaygambetta
Copy link
Member

@jaygambetta jaygambetta commented Dec 23, 2018

Following the instructions, there was a dash missing.

A prompt was not on one of the install lines and since requirements were inline I missed it the first time.

Personally, I would rethink your headings as I don’t know why the using cmake files needs a higher level than the build.

In Terra, we rearranged to be the following headings.

Contributing, Installing Code, Documentation, and Development cycle.

But it builds and works for me now :-).

@atilag
Copy link
Member

atilag commented Dec 26, 2018

Yep, liked that rearranged. We'll change it

@atilag atilag merged commit f5c90c4 into Qiskit:master Dec 26, 2018
eladgoldman pushed a commit to eladgoldman/qiskit-aer that referenced this pull request May 7, 2019
Expectation value of general subset of qubits
dcmckayibm pushed a commit to dcmckayibm/qiskit-aer that referenced this pull request Nov 3, 2019
Patataman added a commit to Patataman/qiskit-aer that referenced this pull request Aug 21, 2023
fix seg fault bc arrays were too big
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