Skip to content

Change array to short syntax in LevelModel#39581

Merged
wilsonge merged 1 commit intojoomla:4.2-devfrom
Digital-Peak:array
Jan 9, 2023
Merged

Change array to short syntax in LevelModel#39581
wilsonge merged 1 commit intojoomla:4.2-devfrom
Digital-Peak:array

Conversation

@laoneo
Copy link
Copy Markdown
Member

@laoneo laoneo commented Jan 9, 2023

Followup of #39297 to change array to short syntax.

Can be merged on review.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@ReLater
Copy link
Copy Markdown
Contributor

ReLater commented Jan 9, 2023

I have tested this item ✅ successfully on 74a6391

Code review


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

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 9, 2023

I have tested this item ✅ successfully on 74a6391


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

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 9, 2023

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 9, 2023
@wilsonge wilsonge merged commit 0788025 into joomla:4.2-dev Jan 9, 2023
@wilsonge wilsonge deleted the array branch January 9, 2023 10:42
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 9, 2023
@wilsonge wilsonge added this to the Joomla! 4.2.7 milestone Jan 9, 2023
@wilsonge
Copy link
Copy Markdown
Contributor

wilsonge commented Jan 9, 2023

Thanks!

charvimehradu pushed a commit to charvimehradu/joomla-cms that referenced this pull request Jan 26, 2023
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.

5 participants