Skip to content

Correct spelling & grammar in 4.4 components/cache/ #16258

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
Dec 16, 2021
Merged

Correct spelling & grammar in 4.4 components/cache/ #16258

merged 1 commit into from
Dec 16, 2021

Conversation

gnito-org
Copy link
Contributor

No description provided.

@carsonbot carsonbot added this to the 4.4 milestone Dec 14, 2021
@@ -7,7 +7,7 @@ Cache Invalidation

Cache invalidation is the process of removing all cached items related to a
change in the state of your model. The most basic kind of invalidation is direct
items deletion. But when the state of a primary resource has spread across
item deletion. But when the state of a primary resource has spread across
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure this change is correct

cc @nicolas-grekas @Nyholm

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It can either be stated as:

  • ... is direct item deletion, or
  • ... is the direct deletion of items.

@@ -47,7 +47,7 @@ you can invalidate the cached items by calling
// if you know the cache key, you can also delete the item directly
$cache->delete('cache_key');

Using tags invalidation is very useful when tracking cache keys becomes difficult.
Using tag invalidation is very useful when tracking cache keys becomes difficult.
Copy link
Contributor

Choose a reason for hiding this comment

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

Unsure here too

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It can either be stated as:

  • Using tag invalidation..., or
  • Using the invalidation of tags...

Copy link
Member

@Nyholm Nyholm left a comment

Choose a reason for hiding this comment

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

Thank you

@javiereguiluz javiereguiluz merged commit bba6c05 into symfony:4.4 Dec 16, 2021
@gnito-org gnito-org deleted the spelling-components-cache-4-4 branch December 16, 2021 19:45
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.

6 participants