Skip to content

WIP: add raycast and logger modules#4

Draft
lu-value wants to merge 5 commits intoVORPCORE:mainfrom
lu-value:feat-raycast-logger
Draft

WIP: add raycast and logger modules#4
lu-value wants to merge 5 commits intoVORPCORE:mainfrom
lu-value:feat-raycast-logger

Conversation

@lu-value
Copy link
Contributor

Pull request template

Important

Please complete all fields. PRs will not be merged if any fields are incomplete. Be respectful, and keep in mind that it may take some time for your PR to be reviewed.

Bear in mind refactors are up to the developers and not it's contributors after all we are the ones giving support when needed. if they are big changes I suggest you to release it under your name instead.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Motive for This Pull Request

This PR is opened to get feedback before the work is finished.

Provide a brief explanation of why these changes are being proposed and what they aim to achieve.

This PR adds a raycast module and a logger module to vorp_lib.

Explain the necessity of these changes and how they will impact the framework or its users.

These changes add reusable utility modules for developers. raycast helps with client-side raycasts and logger provides cleaner logs with resource name, time, and log level.

Please describe the tests you have conducted to verify your changes. Provide instructions so we can reproduce these tests. Also, list any relevant details for your test configuration.

  • Tested with latest vorp scripts
  • Tested with latest artifacts

Notes if any

This PR is still in progress and is not ready to merge yet. It is opened now to receive feedback.

@lu-value lu-value marked this pull request as draft March 24, 2026 23:23
@outsider31000
Copy link
Member

I see this as draft, so make sure you are using methods like the rest of the modules please
Logger:Warn() Logger:Error() etc like you did before.

Right now you doing this as functions in a table. we want to keep the code as consistent as possible just like the rest of the modules. or at least try. Methods are just nicer :) and I have planned features for the logger if it gets merged.

@lu-value
Copy link
Contributor Author

I see this as draft, so make sure you are using methods like the rest of the modules please

Logger:Warn() Logger:Error() etc like you did before.

Right now you doing this as functions in a table. we want to keep the code as consistent as possible just like the rest of the modules. or at least try. Methods are just nicer :) and I have planned features for the logger if it gets merged.

Thanks for the review. I’ll apply the changes !

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.

2 participants