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: changed types and annotations 4 #248

Open
wants to merge 3 commits into
base: refactor/change-types-and-annotations-3
Choose a base branch
from

Conversation

JoaoFerrazfs
Copy link
Contributor

@JoaoFerrazfs JoaoFerrazfs commented Oct 7, 2024

Este PR é o terceiro de um serie de PRs que tem por objetivo adicionar padroes de codigo ao mongolid.

As adições realizadas aqui estão sendo desmebradas do PR:
https://github.com/leroy-merlin-br/mongolid/pull/209/files#diff-44c6c764ba381928e3221e4690c18dc0fe596311af5b123d5d2089ffc4db52af

E são testadas no projeto : https://github.com/JoaoFerrazfs/Mongo-PHP-Docker_BaseProject

@JoaoFerrazfs JoaoFerrazfs force-pushed the refactor/change-types-and-annotations-4 branch 8 times, most recently from 2fd166c to 7bc005b Compare October 7, 2024 18:33
@JoaoFerrazfs JoaoFerrazfs self-assigned this Oct 8, 2024
@JoaoFerrazfs JoaoFerrazfs changed the title refactor: changed types and annotations refactor: changed types and annotations 4 Oct 8, 2024
@JoaoFerrazfs JoaoFerrazfs force-pushed the refactor/change-types-and-annotations-4 branch from 7bc005b to 4ab6c50 Compare October 8, 2024 13:40
@JoaoFerrazfs JoaoFerrazfs marked this pull request as ready for review October 8, 2024 13:51
@JoaoFerrazfs JoaoFerrazfs force-pushed the refactor/change-types-and-annotations-4 branch from 4ab6c50 to dd36ee4 Compare October 8, 2024 14:00
src/LegacyRecord.php Outdated Show resolved Hide resolved
src/Model/AbstractModel.php Outdated Show resolved Hide resolved
@JoaoFerrazfs JoaoFerrazfs force-pushed the refactor/change-types-and-annotations-4 branch from 59c0d22 to b6e8509 Compare October 9, 2024 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants