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: code improvements without affecting logic, part 3. #297

Merged
merged 1 commit into from Jul 23, 2023

Conversation

marcosmarcolin
Copy link
Collaborator

  • Typed returns
  • Micro-optimizations
  • Update some phpdoc comment blocks
  • Remove some unused or unexpected parameters
  • Declare some attributes

As there are no unit tests yet, I tested it in a real application, and it didn't imply in the operation, everything ok.

I would like to test it in real life too.

@walkor walkor merged commit 0ecabe7 into walkor:master Jul 23, 2023
@walkor
Copy link
Owner

walkor commented Jul 23, 2023

Thanks

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.

None yet

2 participants