Skip to content
This repository was archived by the owner on Nov 26, 2017. It is now read-only.

Eliminate check for admin application for using toolbar.#566

Merged
eddieajau merged 2 commits intojoomla:stagingfrom
elinw:patch-2
Dec 2, 2011
Merged

Eliminate check for admin application for using toolbar.#566
eddieajau merged 2 commits intojoomla:stagingfrom
elinw:patch-2

Conversation

@elinw
Copy link
Copy Markdown
Contributor

@elinw elinw commented Nov 28, 2011

No description provided.

@elinw
Copy link
Copy Markdown
Contributor Author

elinw commented Nov 28, 2011

I've asked about this a number of times on the platform list with no substantive response, so I'm just sending the request to take or leave.

@joomla-jenkins
Copy link
Copy Markdown

Unit testing complete. There were 0 failures and 0 errors from 1872 tests and 11090 assertions.
Checkstyle analysis reported 235 warnings and 2 errors.

@joomla-jenkins
Copy link
Copy Markdown

Build triggered by changes to the head.

Unit testing complete. There were 0 failures and 0 errors from 1872 tests and 11090 assertions.
Checkstyle analysis reported 235 warnings and 1 errors.

eddieajau added a commit that referenced this pull request Dec 2, 2011
Eliminate check for admin application for using toolbar.
@eddieajau eddieajau merged commit 6b4a0b5 into joomla:staging Dec 2, 2011
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actually this introduced a code style violation (alert bells ringing ;) )
White space found at end of line...
Please listen to Mr joomla-jenkins ;)

@elinw
Copy link
Copy Markdown
Contributor Author

elinw commented Dec 2, 2011

Funny since there is no net change.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants