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(backend): correctFilename changes dll to exe #11710

Merged
merged 1 commit into from
Aug 12, 2023

Conversation

anatawa12
Copy link
Member

What

.dll.exeになるので.dllのままにする

Why

実行ファイルじゃないのが実行ファイル扱いを受けるので

Additional info (optional)

changelogはファイルアップロード時等にファイル名の拡張子を修正する関数(correctFilename)の挙動を改善に含められると思ったため更新してません

Checklist

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

@github-actions github-actions bot added the packages/backend Server side specific issue/PR label Aug 12, 2023
@github-actions github-actions bot requested review from syuilo and tamaina August 12, 2023 14:45
@codecov
Copy link

codecov bot commented Aug 12, 2023

Codecov Report

Merging #11710 (e9ccd76) into develop (9eccdfb) will decrease coverage by 0.19%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop   #11710      +/-   ##
===========================================
- Coverage    78.67%   78.48%   -0.19%     
===========================================
  Files          922      174     -748     
  Lines        97661    22710   -74951     
  Branches      7705      500    -7205     
===========================================
- Hits         76830    17825   -59005     
+ Misses       20831     4885   -15946     

see 748 files with indirect coverage changes

@tamaina tamaina merged commit 5aada8b into misskey-dev:develop Aug 12, 2023
@tamaina
Copy link
Contributor

tamaina commented Aug 12, 2023

dll配布する場面ってどんな場面やねんと思ったけどとりあえずマージ

@anatawa12 anatawa12 deleted the correct-filename branch August 12, 2023 15:02
anatawa12 added a commit to anatawa12/misskey that referenced this pull request Aug 16, 2023
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.

2 participants