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.

Sign up for GitHub

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

Jump to bottom

Test psi4 on os #120

Merged
merged 22 commits into from
Aug 2, 2021
Merged

Test psi4 on os #120

merged 22 commits into from
Aug 2, 2021

Conversation

dkorzhov
Copy link
Collaborator

No description provided.

@dkorzhov dkorzhov self-assigned this Jul 31, 2021
@dkorzhov dkorzhov mentioned this pull request Jul 31, 2021
Closed
@vvssttkk vvssttkk self-requested a review July 31, 2021 18:14
@vvssttkk vvssttkk linked an issue Jul 31, 2021 that may be closed by this pull request
Решить вопрос с установкой psi4 #113
Closed
@vvssttkk vvssttkk requested a review from SemyonSinchenko July 31, 2021 18:16
@vvssttkk
Copy link
Member

также выполни локально pre-commit и сделай commit изменений

@vvssttkk
Copy link
Member

а что в этом pr делет #118 ? убери, не мешай

@SemyonSinchenko
Copy link
Collaborator

Сложна. Много изменений, завтра гляну.

@dkorzhov
Copy link
Collaborator Author

dkorzhov commented Jul 31, 2021 •
edited
Loading

а что в этом pr делет #118 ? убери, не мешай

сейчас уже сложно разделить, так как изначально в одной репе начал делать, учту на будущее как лучше делать. Посоветуешь, как лучше разделить этот PR?

@vvssttkk
Copy link
Member

а что в этом pr делет #118 ? убери, не мешай

сейчас уже сложно разделить, так как изначально в одной репе начал делать, учту на будущее как лучше делать. Посоветуешь, как лучше разделить этот PR?

вижу что ты у себя делаешь с разных веток. просто в ветку для этого pr убери ненужные файлы. с другим pr та же история

@dkorzhov
Copy link
Collaborator Author

dkorzhov commented Aug 2, 2021

а что в этом pr делет #118 ? убери, не мешай

сейчас уже сложно разделить, так как изначально в одной репе начал делать, учту на будущее как лучше делать. Посоветуешь, как лучше разделить этот PR?

вижу что ты у себя делаешь с разных веток. просто в ветку для этого pr убери ненужные файлы. с другим pr та же история

обновил для двух PR

@vvssttkk vvssttkk changed the base branch from test-psi4-on-os to master August 2, 2021 16:14
@vvssttkk vvssttkk merged commit 1cadd63 into quantum-ods:master Aug 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vvssttkk vvssttkk vvssttkk approved these changes

@SemyonSinchenko SemyonSinchenko SemyonSinchenko approved these changes

Assignees

@dkorzhov dkorzhov

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Решить вопрос с установкой psi4
3 participants
@dkorzhov @vvssttkk @SemyonSinchenko