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

refs #3 CircleCIの導入 #14

Closed
wants to merge 9 commits into from
Closed

Conversation

hamapu
Copy link
Contributor

@hamapu hamapu commented Sep 29, 2019

#3へのPRです

CircleCIでrspecを自動実行するようにする
リポジトリへの権限が足りない場合は付与しますのでコメントください

@hamapu
Copy link
Contributor Author

hamapu commented Sep 30, 2019

[エビデンス] rspecが実行できる:https://circleci.com/gh/sitateru/exam/13

ただ普通にテストに落ちてるので修正しないといけなさそう

@hamapu
Copy link
Contributor Author

hamapu commented Sep 30, 2019

これなんかみてるブランチが違いそうな気がする

https://circleci.com/gh/sitateru/exam/27

↑hamada_m-issue03_3ブランチには無いはずの Issues #02:delete record が実行されている

API叩くときにリビジョン指定できた

@hamapu hamapu changed the title WIP refs #3 CircleCIの導入 refs #3 CircleCIの導入 Sep 30, 2019
@hamapu
Copy link
Contributor Author

hamapu commented Sep 30, 2019

出題当 041070de3b913e293e3c2e24cf39ddf985d148d5 
であればテストが通る
https://circleci.com/gh/sitateru/exam/35

私のPRが入ったところd87f10a
でテストが通らなくなる模様
https://circleci.com/gh/sitateru/exam/37

@tibaraki
ひとまず課題は「circleciでrspec実行」なので、この時点でWIPをはずします
issue #1 に提出したPRの修正が必要そうなので、遡って対応します

This reverts commit dbe41fc.
@tibaraki
Copy link
Contributor

tibaraki commented Oct 1, 2019

課題趣旨に応じた実装はしていただいていると思いますが、revert後にコケちゃってますのでmergeはしないでおきます
もしかすると権限上直しきれない事象かもしれないので、これ以上はできれば程度で大丈夫です

@hamapu
Copy link
Contributor Author

hamapu commented Oct 1, 2019

ありがとうございます。お言葉に甘えて というわけではありませんが

本日(10/2)が期日の認識なので、いったんできるだけ先のissueに取り掛かることにします

@tibaraki tibaraki closed this Oct 29, 2019
@tibaraki tibaraki deleted the hamada_m-issue03 branch October 29, 2019 02:54
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