Skip to content

Conversation

@OskarStark
Copy link
Contributor

Q A
Branch? 7.1
Bug fix? no
New feature? no
Deprecations? no
Issues --
License MIT

@nicolas-grekas
Copy link
Member

It'd be great if you could group your next PRs on the topic in a bigger one 🙏

@OskarStark
Copy link
Contributor Author

Sure, let this one open, I can work on this PR

@OskarStark OskarStark changed the title [Asset] Use CPP Use CPP Jan 2, 2024
@OskarStark OskarStark changed the title Use CPP [Asset][BrowserKit][Cache][Console] Use CPP Jan 2, 2024
@OskarStark OskarStark changed the title [Asset][BrowserKit][Cache][Console] Use CPP [Asset][BrowserKit][Cache][Console][CssSelector] Use CPP Jan 2, 2024
@OskarStark
Copy link
Contributor Author

Can be merged please

@nicolas-grekas
Copy link
Member

Thank you @OskarStark.

@nicolas-grekas nicolas-grekas merged commit 7c53816 into symfony:7.1 Jan 4, 2024
@OskarStark OskarStark deleted the asset-cpp branch January 4, 2024 11:01
nicolas-grekas added a commit that referenced this pull request Jan 8, 2024
This PR was merged into the 7.1 branch.

Discussion
----------

[Cache] revert property introduction

| Q             | A
| ------------- | ---
| Branch?       | 7.1
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        |
| License       | MIT

This property sneaked into in #53347 (probably by mistake).

Commits
-------

066420f revert property introduction
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