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

feat(meme): 短いオプションのキーが使えるように #586

Merged
merged 1 commit into from
Nov 26, 2022

Conversation

MikuroXina
Copy link
Collaborator

Type of Change:

Meme の機能追加

Details of implementation (実施内容)

MemeTemplateoptionsKeysc などの一文字だけを使ったり flagsKeysverbose などの 2 文字以上の文字列を指定しても正しく動作するようになりました.

Additional Information (追加情報)

#585 をより簡潔にするための機能です. なるべく早くレビューが欲しいです.

@MikuroXina MikuroXina self-assigned this Nov 26, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #586 (82dd3a0) into main (e502323) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #586      +/-   ##
==========================================
+ Coverage   73.84%   73.86%   +0.02%     
==========================================
  Files          63       63              
  Lines        4531     4535       +4     
  Branches      401      402       +1     
==========================================
+ Hits         3346     3350       +4     
  Misses        785      785              
  Partials      400      400              
Flag Coverage Δ
unittests 73.86% <100.00%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/service/command/meme.ts 76.69% <100.00%> (+0.94%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@m1sk9 m1sk9 left a comment

Choose a reason for hiding this comment

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

LGTM ❄️

@MikuroXina MikuroXina merged commit 66205ab into main Nov 26, 2022
@MikuroXina MikuroXina deleted the feat/short-option-key branch November 26, 2022 08:52
raiga0310 added a commit that referenced this pull request Nov 26, 2022
* feat: オプションの追加

* オプション追加実装に伴うテストケースの追加

* chore: テストの引数の変更

* refactor: オプション引数への再設計( #586 対応待ち )

* fix: オプションの設定もれの修正

* chore: テストケースの修正

* chore: deletion comment

Co-authored-by: Mikuroさいな <ryosukadnak@gmail.com>

* fix: テストケースの追加 修正

* refactor: オプション・フラグの重複時への対応

* feat: 引数不足時のテストケースの追加

* chore: ヘルプメッセージの親切化

Co-authored-by: m2en <me@m2en.dev>

Co-authored-by: Mikuroさいな <ryosukadnak@gmail.com>
Co-authored-by: m2en <me@m2en.dev>
This was referenced Nov 28, 2022
This was referenced Sep 18, 2023
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.

3 participants