Skip to content

Remove unused deprecated functions from OCP\JSON#8879

Merged
rullzer merged 2 commits intomasterfrom
dep_json
Mar 19, 2018
Merged

Remove unused deprecated functions from OCP\JSON#8879
rullzer merged 2 commits intomasterfrom
dep_json

Conversation

@rullzer
Copy link
Copy Markdown
Member

@rullzer rullzer commented Mar 19, 2018

No description provided.

rullzer added 2 commits March 19, 2018 11:30
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Copy link
Copy Markdown
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

🔥

Copy link
Copy Markdown
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

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

Did you make sure it's not called internally in our private code?

@rullzer
Copy link
Copy Markdown
Member Author

rullzer commented Mar 19, 2018

phpstorm told me no.

@MorrisJobke
Copy link
Copy Markdown
Member

Added them also to #7827 ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants