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

Add CONTRIBUTING.md #207

Merged
merged 1 commit into from Jun 19, 2020
Merged

Add CONTRIBUTING.md #207

merged 1 commit into from Jun 19, 2020

Conversation

y-yu
Copy link
Contributor

@y-yu y-yu commented Dec 30, 2019

  • Qulacs build and test phases are turning complicated.
    • make, mate test, make pythontest, python setup.py install etc...
  • So I add CONTRIBUTING.md which says the way how to bulid, format (Add formatter #201 ) and test.
  • I want to discuss if other things are needed to be written in CONTRIBUTING.md or not 🤔

CONTRIBUTING.md Outdated
### Tips

- We only check the format in Linux
so Travis CI could point out some format errors even if there is nothing error in your Windows or macOS.
Copy link
Contributor

Choose a reason for hiding this comment

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

We only check the format in Linux.
Travis CI could point out format errors even if there are no errors in your Windows or macOS.

が正しい英語かなー?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Would it be better to use there is no error instead of there are no errors?

Copy link
Contributor

Choose a reason for hiding this comment

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

そう変更してください!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry for my very very late fix 🙇

@corryvrequan
Copy link
Contributor

LGTM!

@corryvrequan corryvrequan merged commit 973b5b4 into qulacs:dev Jun 19, 2020
@y-yu y-yu deleted the add-contributing branch June 19, 2020 11:58
heliac2000 pushed a commit to heliac2000/qulacs that referenced this pull request Aug 21, 2020
m-ymzk added a commit to m-ymzk/qulacs that referenced this pull request Apr 16, 2023
[Refactoring] named-gateのOpenMP関係の関数を共通化
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