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(Lua): Better error messages #639

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

feat(Lua): Better error messages #639

wants to merge 2 commits into from

Conversation

UE4SS
Copy link
Collaborator

@UE4SS UE4SS commented Aug 28, 2024

Description

Improves Lua error messages.

Type of change

Please delete options that are not relevant.

  • Other... Please describe: Non-breaking improvement

How Has This Been Tested?

TBD

Checklist

Please delete options that are not relevant. Update the list as the PR progresses.

  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • I have added the necessary description of this PR to the changelog, and I have followed the same format as other entries.
  • Any dependent changes have been merged and published in downstream modules.

Additional context

This PR is WIP, do not merge yet.

Copy link
Contributor

github-actions bot commented Aug 28, 2024

MSVC-Game__Debug__Win64 Download Logs
Build Details
Name Information
PR Commit 175d86e
Merge Commit bdf96b9
Size 44.29 MB
Last Updated Aug 28, 24, 8:11:40 PM UTC
Expires At Sep 11, 24, 8:11:36 PM UTC

MSVC-Game__Shipping__Win64 Download Logs
Build Details
Name Information
PR Commit 175d86e
Merge Commit bdf96b9
Size 25.73 MB
Last Updated Aug 28, 24, 8:13:13 PM UTC
Expires At Sep 11, 24, 8:13:11 PM UTC

@UE4SS
Copy link
Collaborator Author

UE4SS commented Aug 28, 2024

This PR is likely to change drastically, I'm using it just to build right now because I can't build locally.
Will rebase in the end to get rid of any random commits.

@UE4SS UE4SS marked this pull request as draft September 2, 2024 20:27
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