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

Add setter for defaultSchema propierty. #743

Closed
wants to merge 1 commit into from

Conversation

terabytesoftw
Copy link
Member

Q A
Is bugfix? ✔️
New feature?
Breaks BC?

@what-the-diff
Copy link

what-the-diff bot commented Aug 15, 2023

PR Summary

  • Introduction of a New Method in AbstractSchema.php and SchemaInterface.php
    The method defaultSchema(string $defaultSchema): void has been added to both the AbstractSchema.php and SchemaInterface.php files. This method enables the setting up of predefined schema for certain operations, alleviating the need to repeat the same steps each time a new procedure is conducted.

  • Test for New Method in SchemaTest.php
    A new test method defaultSchema() has been introduced in SchemaTest.php. This is designed to ensure that the new defaultSchema method functions as intended, providing accurate and reliable results. Consequently, any potential issues could be identified and rectified as quickly as possible, ensuring the system's overall stability and a smoother user-experience.

@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (b047dae) 99.73% compared to head (64aaf0c) 99.73%.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #743   +/-   ##
=========================================
  Coverage     99.73%   99.73%           
- Complexity     1269     1270    +1     
=========================================
  Files            63       63           
  Lines          3038     3040    +2     
=========================================
+ Hits           3030     3032    +2     
  Misses            8        8           
Files Changed Coverage Δ
src/Schema/AbstractSchema.php 99.42% <100.00%> (+<0.01%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@terabytesoftw terabytesoftw marked this pull request as draft August 15, 2023 14:16
@terabytesoftw terabytesoftw removed the type:bug Bug label Aug 15, 2023
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