Skip to content

Apply fixes from StyleCI #144

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 2, 2023
Merged

Apply fixes from StyleCI #144

merged 1 commit into from
May 2, 2023

Conversation

mechelon
Copy link
Member

@mechelon mechelon commented May 2, 2023

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.
@mechelon mechelon merged commit fe0707c into feature/database-creation May 2, 2023
@mechelon mechelon deleted the analysis-O3Re50 branch May 2, 2023 03:50
@what-the-diff
Copy link

what-the-diff bot commented May 2, 2023

PR Summary

  • Improved documentation for DatabaseBuilder constructor
    Added PHPDoc to make the constructor more understandable
  • Added documentation for createInPage method
    Enhanced code readability by including PHPDoc for the method
  • Updated database creation title
    Changed the default title from "Database" to a customizable $title in the payload() method
  • Cleaned up code and added future considerations
    Removed unused code (inline()) and included a @todo comment for potential updates with Notion API version increase
  • Corrected typos in PropertyBuilder.php
    Fixed typos to ensure accuracy and readability
  • Enhanced Databases class and methods documentation
    Added missing documentation for better understanding and maintainability
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@mechelon @StyleCIBot