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

refactor(backend): validateNoteの引数の型を強くし、anyを除去 #10325

Merged
merged 3 commits into from
Apr 14, 2023
Merged

refactor(backend): validateNoteの引数の型を強くし、anyを除去 #10325

merged 3 commits into from
Apr 14, 2023

Conversation

KisaragiEffective
Copy link
Collaborator

What

validateNoteメソッドからanyを取り除きました。

Why

#3622

Additional info

議論の余地があると思われる変更点

  • 8622100 の変更内容
    • アサーションの内容から推論して、actualの内容を変更しました。変更後のコードは意図された内容でしょうか?

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests
    • not relevant

@github-actions github-actions bot added the packages/backend Server side specific issue/PR label Mar 14, 2023
@KisaragiEffective KisaragiEffective changed the title refactor(backend): validateNoteからanyを取り除く refactor(backend): validateNoteの引数の型を強くし、anyを除去 Mar 14, 2023
@codecov
Copy link

codecov bot commented Mar 14, 2023

Codecov Report

Merging #10325 (9f86e09) into develop (c05c504) will increase coverage by 20.30%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop   #10325       +/-   ##
============================================
+ Coverage    74.41%   94.71%   +20.30%     
============================================
  Files          872        2      -870     
  Lines        85156      454    -84702     
  Branches      5599       59     -5540     
============================================
- Hits         63367      430    -62937     
+ Misses       21789       24    -21765     

see 874 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Co-authored-by: Acid-Chicken <root@acid-chicken.com>
@tamaina
Copy link
Contributor

tamaina commented Apr 14, 2023

テストが長い

@tamaina tamaina merged commit 83d0f81 into misskey-dev:develop Apr 14, 2023
@tamaina
Copy link
Contributor

tamaina commented Apr 14, 2023

🙏 🙏🙏

na2na-p pushed a commit to na2na-p/misskey that referenced this pull request May 10, 2023
* refactor(backend): validateNoteの引数の型を推論する

* fix(backend): アサーションの内容から推論してエラーの内容を期待されるであろう式へと変更する

* refactor

Co-authored-by: Acid-Chicken <root@acid-chicken.com>

---------

Co-authored-by: tamaina <tamaina@hotmail.co.jp>
Co-authored-by: Acid-Chicken <root@acid-chicken.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend Server side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants