8000 [Cache] Fix outdated CHANGELOG line by ogizanagi · Pull Request #28922 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[Cache] Fix outdated CHANGELOG line #28922

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
merged 1 commit into from
Oct 19, 2018
Merged

[Cache] Fix outdated CHANGELOG line #28922

merged 1 commit into from
Oct 19, 2018

Conversation

ogizanagi
Copy link
Contributor
Q A
Branch? master
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets N/A
License MIT
Doc PR N/A

Unless it was kept on purpose, for Symfony\Contracts\Cache\CacheInterface discoverability? In which case I'd add the FQCN.

@@ -7,7 +7,6 @@ CHANGELOG
* added support for connecting to Redis clusters via DSN
* added support for configuring multiple Memcached servers via DSN
* added `MarshallerInterface` and `DefaultMarshaller` to allow changing the serializer and provide one that automatically uses igbinary when available
* added `CacheInterface`, which provides stampede protection via probabilistic early expiration and should become the preferred way to use a cache
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or just replace "added" by "implemented" to hint about it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as well :)
Done.

@ogizanagi ogizanagi changed the title [Cache] Remove outdated CHANGELOG line [Cache] Fix outdated CHANGELOG line Oct 19, 2018
@fabpot
Copy link
Member
fabpot commented Oct 19, 2018

Thank you @ogizanagi.

@fabpot fabpot merged commit 4e89a4b into symfony:master Oct 19, 2018
fabpot added a commit that referenced this pull request Oct 19, 2018
This PR was merged into the 4.2-dev branch.

Discussion
----------

[Cache] Fix outdated CHANGELOG line

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no <!-- don't forget to update src/**/CHANGELOG.md files -->
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no <!-- don't forget to update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets | N/A   <!-- #-prefixed issue number(s), if any -->
| License       | MIT
| Doc PR        | N/A

Unless it was kept on purpose, for `Symfony\Contracts\Cache\CacheInterface` discoverability? In which case I'd add the FQCN.

Commits
-------

4e89a4b [Cache] Fix outdated CHANGELOG line
@ogizanagi ogizanagi deleted the cache/rm_outdated_changelog branch October 19, 2018 11:43
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
0