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

fix: importsNotUsedAsValues を error に変更 #431

Merged
merged 2 commits into from
Aug 26, 2022

Conversation

MikuroXina
Copy link
Collaborator

Type of Change:

tsconfig の変更

Cause of the Problem (問題の原因)

これまでコードレビューにおいて import type を指摘していましたが, これでは作業効率が少し悪くなっていました.

Details of implementation (実施内容)

tsconfig.json にて importsNotUsedAsValueserror に設定し, 値を import していない場合の通常の import はエラーになるようにしました.

@MikuroXina MikuroXina requested a review from m1sk9 August 26, 2022 11:08
@codecov-commenter
Copy link

codecov-commenter commented Aug 26, 2022

Codecov Report

Merging #431 (40b28c6) into main (3b12150) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #431   +/-   ##
=======================================
  Coverage   71.36%   71.37%           
=======================================
  Files          58       58           
  Lines        3932     3933    +1     
  Branches      369      369           
=======================================
+ Hits         2806     2807    +1     
  Misses        757      757           
  Partials      369      369           
Flag Coverage Δ
unittests 71.37% <100.00%> (+<0.01%) ⬆️

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

Impacted Files Coverage Δ
src/service/command/role-rank.ts 79.16% <100.00%> (+0.44%) ⬆️

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

@MikuroXina MikuroXina self-assigned this Aug 26, 2022
@MikuroXina MikuroXina marked this pull request as ready for review August 26, 2022 11:08
@MikuroXina MikuroXina enabled auto-merge (squash) August 26, 2022 11:08
@MikuroXina MikuroXina changed the title fix: Make importsNotUsedAsValues error fix: importsNotUsedAsValues を error に変更 Aug 26, 2022
@MikuroXina MikuroXina merged commit f8b7fd2 into main Aug 26, 2022
@MikuroXina MikuroXina deleted the fix/force-import-type branch August 26, 2022 13:28
@github-actions
Copy link
Contributor

🎉 This PR is included in version 1.25.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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