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

Use dev version of Yii Data + increase psalm level to 1 #35

Merged
merged 1 commit into from
Nov 21, 2023
Merged

Conversation

vjik
Copy link
Member

@vjik vjik commented Nov 21, 2023

Q A
Is bugfix?
New feature?
Breaks BC?
Fixed issues -

Copy link

what-the-diff bot commented Nov 21, 2023

PR Summary

  • Updated the stability and package version in Composer Configuration
    The 'composer.json' file, which controls dependencies for the project, has been updated. Its 'minimum-stability' setting was changed to 'dev' allowing us to use packages in their developmental stages. The 'prefer-stable' was set to true, which means we prefer stable versions of dependencies where possible. Additionally, the 'yiisoft/data' package version has been updated.

  • Adjusted the Error Level in Psalm Configuration
    The 'psalm.xml' file, controlling a static analysis tool used to identify errors in the code, was changed. It had its 'errorLevel' modified to '1', which represents a stricter level of error checking in our code.

  • Improved Documentation in Abstract Query Data Reader
    The 'AbstractQueryDataReader.php' script had been updated to include some annotations related to 'psalm'. This update improves the documentation for developers, especially for the 'prepareHandlers' and 'read' methods.

  • Removed the Format Values Method
    The 'formatValues' method was removed from 'Filter/CompareFilter.php'. This indicates a change in how we process and format data in our filters.

@vjik vjik merged commit b0052d1 into master Nov 21, 2023
27 of 28 checks passed
@vjik vjik deleted the adapt branch November 21, 2023 12:05
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