Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Tests] Remove $this occurrences in future static data providers #48980

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

Merged

Conversation

alexandre-daubois
Copy link
Member

Q A
Branch? 5.4
Bug fix? yes-ish
New feature? no
Deprecations? no
Tickets Easing #48668 (comment)
License MIT
Doc PR NA

@carsonbot carsonbot added this to the 5.4 milestone Jan 14, 2023
@alexandre-daubois alexandre-daubois force-pushed the fix/remove-this-data-provider-1 branch 2 times, most recently from c0b9c73 to 99f3168 Compare January 14, 2023 18:53
@alexandre-daubois alexandre-daubois force-pushed the fix/remove-this-data-provider-1 branch from 99f3168 to 6fb7cb9 Compare January 14, 2023 19:14
@OskarStark
Copy link
Contributor

Friendly ping @nicolas-grekas this is another PR on the road to static dataProviders. Thanks

@fabpot
Copy link
Member

fabpot commented Jan 15, 2023

Thank you @alexandre-daubois.

@fabpot fabpot merged commit 4ae23e6 into symfony:5.4 Jan 15, 2023
@alexandre-daubois alexandre-daubois deleted the fix/remove-this-data-provider-1 branch January 15, 2023 17:58
nicolas-grekas added a commit that referenced this pull request Feb 1, 2023
… future static data providers (alexandre-daubois)

This PR was merged into the 5.4 branch.

Discussion
----------

[Tests] New iteration of removing `$this` occurrences in future static data providers

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | yes-ish
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | Easing #48668 (comment)
| License       | MIT
| Doc PR        | _NA_

Follow-up of #48980

Commits
-------

1386ac2 [Tests] New iteration of removing `$this` occurrences in future static data providers
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.

5 participants