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

Travis CIからGithub Actionsに変更 #7

Merged
merged 3 commits into from
Mar 11, 2022
Merged

Conversation

imaxyz
Copy link
Contributor

@imaxyz imaxyz commented Mar 9, 2022

目的

  • toxを、Travis CIから実行するのをやめて、Github Actionsから実行するように変更する

作業内容

  • Travis CIの使用を止めて、Github Actionsでtoxを実行するように変更しました。
    • Travis CIの設定ファイルを削除しました。
    • tox.iniに、Github Actionsの設定を追記しました。

レビューしてほしいところ

  • .github/workflows/tests.yml の書き方は問題ないか
  • tox.iniの書き方に不足はないか
  • コメントに違和感はないか
  • バージョン1.3をリリースするにあたって、何か他に変更が必要な点はあるか

@imaxyz imaxyz marked this pull request as ready for review March 10, 2022 05:31
Copy link
Member

@shimizukawa shimizukawa left a comment

Choose a reason for hiding this comment

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

LGTM!!

@@ -115,3 +115,11 @@ deps =
flake8
commands =
flake8 bpmappers/

Copy link
Member

Choose a reason for hiding this comment

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

メモ: 編集範囲外の [testenv:py37-dj31] 等のセクションもまとめられそうですが、またいずれ。

Copy link
Collaborator

@tokibito tokibito left a comment

Choose a reason for hiding this comment

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

LGTM

# push と pull request イベント毎にGtihubのWorkflowを起動する
on:
- push
- pull_request

Choose a reason for hiding this comment

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

pushとpull_requestで絞り込みせずに起動させた場合pull_request時に2回同じテストが実行されてしまいます。

なのでpull_requestのopened, syncronize & push masterにするかpushのみに条件を絞った方が良いです。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

同じテストが同時に2回走ってるぽいのは気になってました。
検討します。

Copy link
Contributor Author

@imaxyz imaxyz Mar 11, 2022

Choose a reason for hiding this comment

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

いったん issueにしました #8

別PRで対応できればと思います

@imaxyz imaxyz merged commit 65153b7 into master Mar 11, 2022
@imaxyz imaxyz deleted the travis_ci-to-github_actions branch March 11, 2022 07:31
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.

4 participants