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

NinjaSlayer: NJコマンドでシークレットロールできないのを修正する #126

Merged
merged 1 commit into from
Feb 23, 2020

Conversation

ochaochaocha3
Copy link
Member

NinjaSlayerのNJコマンドでシークレットロールできないのを修正しました。changeText で参照する正規表現において、シークレットロールを示す先頭の S を考慮していませんでした。

シークレットロールを示す先頭の「S」を考慮していなかった
@ochaochaocha3 ochaochaocha3 added the bug バグってる! label Feb 22, 2020
@ochaochaocha3 ochaochaocha3 self-assigned this Feb 22, 2020
@codecov-io
Copy link

codecov-io commented Feb 22, 2020

Codecov Report

Merging #126 into master will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #126      +/-   ##
=========================================
+ Coverage   86.07%   86.1%   +0.03%     
=========================================
  Files         188     188              
  Lines       21826   21907      +81     
=========================================
+ Hits        18786   18863      +77     
- Misses       3040    3044       +4
Impacted Files Coverage Δ
diceBot/NinjaSlayer.rb 98.92% <0%> (+0.02%) ⬆️
diceBot/Cthulhu_ChineseTraditional.rb 14.91% <0%> (ø) ⬆️
diceBot/Cthulhu7th_ChineseTraditional.rb 15.3% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6dc09be...5d1274f. Read the comment docs.

Copy link
Member

@ysakasin ysakasin left a comment

Choose a reason for hiding this comment

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

LGTM!

@ysakasin ysakasin merged commit ccdbe3f into master Feb 23, 2020
@ysakasin ysakasin deleted the NinjaSlayer-fix_secret_roll branch February 23, 2020 15:43
ysakasin added a commit that referenced this pull request Aug 30, 2020
NinjaSlayer: NJコマンドでシークレットロールできないのを修正する
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug バグってる!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants