-
Notifications
You must be signed in to change notification settings - Fork 71
Support DLang #145
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
Support DLang #145
Conversation
こんにちは @penpenpng さん! PRありがとうございます。
を実行すればテストが走るはずです。 やることは確か以下の通りです。あとでもっかいみなおします。すみません。 [2] テストの時にコンパイラとコンパイルを必要とするので、以下の作業を行ってください。
|
アドバイスありがとうございます! |
Codecov Report
@@ Coverage Diff @@
## master #145 +/- ##
==========================================
+ Coverage 89.24% 89.57% +0.32%
==========================================
Files 51 52 +1
Lines 2353 2465 +112
==========================================
+ Hits 2100 2208 +108
- Misses 253 257 +4
Continue to review full report at Codecov.
|
Cool! かなりいい感じです! CRします! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ほぼいい感じです。あとはテストを追加してくださればマージします。
すみません。最近のnim言語の追加でmerge conflictが起きてしまったので、修正をおねがいします。 |
丁寧に案内していただきありがとうございました! |
恥ずかしながらテストの仕組みがよく分からなかったので、テストコードに手を付けられていません。
必要ならばテストケースの追加作業も行いますので、アドバイスいただけないでしょうか?