Skip to content

[4.0] fix js code style#30335

Merged
richard67 merged 1 commit intojoomla:4.0-devfrom
brianteeman:drone
Aug 10, 2020
Merged

[4.0] fix js code style#30335
richard67 merged 1 commit intojoomla:4.0-devfrom
brianteeman:drone

Conversation

@brianteeman
Copy link
Contributor

@brianteeman brianteeman commented Aug 10, 2020

code review only

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev Test instructions missing labels Aug 10, 2020
@joomla-cms-bot
Copy link

Please add more information to your issue. Without test instructions and/or any description we will close this issue within 4 weeks. Thanks.
This is an automated message from the J!Tracker Application.

@richard67
Copy link
Member

Fixes #30334 (comment) .

@richard67 richard67 merged commit 83742b2 into joomla:4.0-dev Aug 10, 2020
@richard67 richard67 added this to the Joomla 4.0 milestone Aug 10, 2020
@richard67
Copy link
Member

Merged by review. It fixes the javascript cs errors which will be reported like here https://ci.joomla.org/joomla/joomla-cms/34613/1/25 when the eslint package will be updated to the latest version.

@brianteeman brianteeman deleted the drone branch August 10, 2020 15:37
dgrammatiko added a commit to dgrammatiko/joomla-cms that referenced this pull request Aug 13, 2020
…om_templates

* '4.0-dev' of github.com:joomla/joomla-cms:
  Add closing quote (joomla#30353)
  [4.0] Make sure the renderer does not manipulate the inline CSS and JS (joomla#28719)
  [4.0] Atum and Cassiopea Change alerts styling - space saving + (joomla#30294)
  Update editor versions (joomla#30340)
  [4.0][mod_menu] Add space for 'menu class' (joomla#30341)
  Optimize code for aria-current (joomla#30328)
  [4.0] Composer and npm updates 10 Aug (joomla#30334)
  [4.0] fix js code style (joomla#30335)
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester Test instructions missing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments