Skip to content

7.1.1 #182

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.

Sign up for GitHub

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

Jump to bottom
Merged
mmelograno merged 36 commits into from
Mar 28, 2022
Merged

7.1.1 #182

mmelograno merged 36 commits into from
Mar 28, 2022

Conversation

@mmelograno
Copy link
Contributor

@mmelograno mmelograno commented Mar 25, 2022

PHP SDK

What did you accomplish?

  • Removed unused logic for HTTP and requests dependency.
  • Removed all producer logic for Storages.
  • Added SafeRedisWrapper for handling exception in Redis.
  • Fixed typos in bucket and CONTRIBUTORS-GUIDE.
  • Added .gitattributes for excluding tests/.

How do we test the changes introduced in this PR?

Extra Notes

Matias Melograno and others added 30 commits February 11, 2022 10:41
removed http logic from repo
238053f
removed import
29c0cc1
removed Process component
0acf0b2
updated version for test
1d8e593
added error handling on manager
a4cf6cd
rollback changes in manager of err handling
09157fe
added wrapper exception for PHP Redis
11f4606
renamed Wrapper
713601b
removed commented lines on imports
215f943
cleaning SplitCache
6e25937
@mmelograno
Merge pull request #174 from splitio/task/requestDepClen
d72b8a6 
Task/request dep clen
Merge branch 'develop' of github.com:splitio/php-client into task/red…
cedc2be 
…isWrapper
cleaned segmentCache
98c0a08
cleaned pool
8690187
added cleaning for key created
4cf0a84
@mmelograno
Merge pull request #176 from splitio/task/cleanSplitCache
7d75ea5 
Task/clean split cache
Merge branch 'task/redisWrapper' of github.com:splitio/php-client int…
d0f73b4 
…o task/cleanSegmentsCache
@mmelograno
Merge pull request #177 from splitio/task/cleanSegmentsCache
97765c1 
Task/clean segments cache
added missing methods
7ffc5e4
@mmelograno
Merge pull request #178 from splitio/task/cleanPool
815673b 
Task/clean pool
added comments
49fed8b
prepare release for stage
24e38e3
returning Item when an exception occurred
9ed72d1
updated tests with changes in redisWrapper
bd52908
@mmelograno
removed commented test
f85782e
@mmelograno
remove Item
bb602d3
@mmelograno
renamed cache methods
c436e33
@mmelograno
Merge pull request #179 from splitio/task/removeItem
ddcb283 
Task/remove item
@mmelograno
added .gitattributes to exclude tests
12ddb5c
@mmelograno
updated changelog
efaf54f
@mmelograno mmelograno changed the title Develop 7.1.1 Mar 25, 2022
@sonarqube-pull-requests
Copy link

sonarqube-pull-requests bot commented Mar 28, 2022

SonarQube Quality Gate

Quality Gate failed

Failed condition 0.0% 0.0% Coverage on New Code (is less than 80%)

See analysis details on SonarQube

@mmelograno mmelograno merged commit e04ec97 into master Mar 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mredolatti mredolatti mredolatti approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mmelograno @mredolatti