Skip to content

Conversation

@frankmayer
Copy link
Contributor

@frankmayer frankmayer commented Dec 15, 2016

Summary of Changes

  • Replace unnecessary double quotes

This PR is part of a set to try to separate some of the changes done in one of my previous batch PR's for the libraries/cms directory, which is still on hold (#12171).
Once the new set is merged it will hopefully reduce the changes in that PR, so it can be reviewed easier and finally be merged.

The changes in this PR should be fairly easy to review. In hope that this will get merged quickly. ;)

Testing Instructions

None, should not change behavior

Documentation Changes Required

None.

@C-Lodder
Copy link
Member

I have tested this item ✅ successfully on 0aec41f

based on code review


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13235.

@yvesh
Copy link
Member

yvesh commented Dec 16, 2016

I have tested this item ✅ successfully on 0aec41f

code review


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13235.

@jeckodevelopment
Copy link
Member

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13235.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Dec 16, 2016
@jeckodevelopment jeckodevelopment added this to the Joomla 3.7.0 milestone Dec 16, 2016
@rdeutz rdeutz merged commit 88014d2 into joomla:staging Dec 16, 2016
@joomla-cms-bot joomla-cms-bot added PR-staging and removed RTC This Pull Request is Ready To Commit labels Dec 16, 2016
@frankmayer frankmayer deleted the unnecessary-double-quotes-in-libraries-cms branch December 25, 2016 21:46
@frankmayer frankmayer mentioned this pull request Jan 5, 2017
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants