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

CI: run rspec in GitHub Actions #17

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

YusukeHosonuma
Copy link
Collaborator

@YusukeHosonuma YusukeHosonuma commented Feb 16, 2020

GitHub Actions で Rspec も実行するようにしました。

masterブランチにマージされた時でも実行されるように、pushトリガーで RSpec を実行するワークフローrspec.ymlという名前で用意しました。なお、既存のワークフロー名はrubocop.ymlに名前を変更しています。
image

ついでに README の見栄えも良くしてみました(気に入らなければ discard で大丈夫です)。
image

TODO

  • main.yml -> rubocop.yml
  • rspec.ymlの作成(Pushトリガーで Rspec を実行)
  • README に各種バッジを追加

Comment on lines +27 to +29
- name: Run test
run: |
rake test
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ここ以外はmain.ymlからコピーしてきているので変更はありません。

@@ -1,5 +1,9 @@
# AutoReserve

[![Test](https://github.com/po-miyasaka/AutoReserve/workflows/Test/badge.svg)](https://github.com/po-miyasaka/AutoReserve/actions?query=workflow%3ATest)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Actions タブ > Create status badge から作成しています。
image

Comment on lines +4 to +5
[![License](https://img.shields.io/github/license/po-miyasaka/AutoReserve)](https://github.com/po-miyasaka/AutoReserve/blob/master/LICENSE)
[![Twitter](https://img.shields.io/twitter/url?style=social&url=https%3A%2F%2Ftwitter.com%2Fpo_miyasaka)](https://twitter.com/po_miyasaka)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@@ -1,4 +1,4 @@
name: Run danger
name: Rubocop
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

どういう命名にするかは少し迷ったのですが、ワークフローの一覧に表示されて直感的と思える名称にしました(ファイル名もそれに合わせています)。
image

@po-miyasaka
Copy link
Owner

あざます!!!!めちゃくちゃありがたやなプルリク😭

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