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

Packaging #1

Merged
merged 7 commits into from Feb 17, 2022
Merged

Packaging #1

merged 7 commits into from Feb 17, 2022

Conversation

eggplants
Copy link
Contributor

@eggplants eggplants commented Feb 17, 2022

主に開発環境周りです。

  • パッケージ化し依存ライブラリのインストールやコマンドwicafeとして使いやすく
  • style checker CI の追加
  • release CIの追加
    • git tag <version> && git push --tagsで動作
    • 必須: PYPI_USERNAMEPYPI_PASSWORDをsecretsに設定

@puhitaku puhitaku changed the base branch from master to pr/eggplants February 17, 2022 15:07
@puhitaku
Copy link
Owner

puhitaku commented Feb 17, 2022

PR ありがとうございます。いくつかの理由によりこの PR はそのまま merge できません。
一言で書くと、事前に相談が欲しかったです。CONTRIBUTING を書いていないため後出しになってしまう点申し訳ありませんが、このような個人のスタイル差が出やすい変更は一般に事前に issue を書くことが望ましいです。

  • .gitignore は gitignore.io で生成しているため典拠の異なる gitignore には変更しないでください
  • Markdownlint の標準構成に沿う修正をされていますが、私の Markdown のスタイルと異なる上に構成ファイルが存在しません(暗示的なツールへの依存は望ましくない)
  • CI 周辺は基本的にありがたい & 今回は偶然私のパッケージングスタイルに近いですが、基本的には先に相談してください

よって、今回の提出内容は別な branch に merge し、そこから master に cherry-pick する形で merge したいと思います。ありがとうございました。

@puhitaku puhitaku merged commit c3b0b4f into puhitaku:pr/eggplants Feb 17, 2022
@eggplants eggplants deleted the packaging branch February 17, 2022 15:16
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

2 participants