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(skyrim-platform): revert "temporary duplicate dlls in root folder" #2320

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

nic11
Copy link
Collaborator

@nic11 nic11 commented Feb 7, 2025

Reverts #2316


Important

Reverts duplicate DLL handling in index.js and updates DistContentsExpected.json to remove fmt.dll and spdlog.dll from the client directory.

  • Behavior:
    • Reverts handling of requiredVcpkgDllsRoot in index.js, removing duplication of fmt.dll and spdlog.dll in the root folder.
  • Distribution:
    • Updates DistContentsExpected.json to remove fmt.dll and spdlog.dll from the client directory.

This description was created by Ellipsis for 81fe7b8. It will automatically update as commits are pushed.

@nic11 nic11 changed the title Revert "feat(skyrim-platform): temporary duplicate dlls in root folder" feat(skyrim-platform): revert "temporary duplicate dlls in root folder" Feb 7, 2025
@nic11 nic11 marked this pull request as ready for review February 7, 2025 18:37
@nic11 nic11 merged commit 528a489 into main Feb 7, 2025
8 of 16 checks passed
@nic11 nic11 deleted the revert-2316-dllsroot branch February 7, 2025 18:37
Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 81fe7b8 in 26 seconds

More details
  • Looked at 43 lines of code in 2 files
  • Skipped 0 files when reviewing.
  • Skipped posting 4 drafted comments based on config settings.
1. skyrim-platform/tools/dev_service/index.js:7
  • Draft comment:
    Removal of duplicate DLL copy for fmt.dll and spdlog.dll; ensure that spdlog.dll is correctly deployed elsewhere if needed.
  • Reason this comment was not posted:
    Comment was not on a location in the diff, so it can't be submitted as a review comment.
2. unit/DistContentsExpected.json:332
  • Draft comment:
    Updated expected files: removed client/fmt.dll and client/spdlog.dll to match reverted behavior.
  • Reason this comment was not posted:
    Comment did not seem useful. Confidence is useful = 0% <= threshold 50%
    This comment is purely informative, stating that some files were removed to match reverted behavior. It doesn't provide any actionable feedback or suggestions for improvement.
3. skyrim-platform/tools/dev_service/index.js:7
  • Draft comment:
    Ensure removal of duplicate DLL copies (fmt.dll and spdlog.dll) is intentional and won’t break dependency resolution.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50%
    None
4. unit/DistContentsExpected.json:332
  • Draft comment:
    Test expectations updated: confirm that removal of 'client/fmt.dll' and 'client/spdlog.dll' from root is correct.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50%
    None

Workflow ID: wflow_kuJtfBRMQuQegZ1Q


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

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.

1 participant