Skip to content

remove index.html from all folders b... through i...#5380

Merged
wilsonge merged 2 commits intojoomla:stagingfrom
okonomiyaki3000:kill-indexex-b-i
Dec 10, 2014
Merged

remove index.html from all folders b... through i...#5380
wilsonge merged 2 commits intojoomla:stagingfrom
okonomiyaki3000:kill-indexex-b-i

Conversation

@okonomiyaki3000
Copy link
Copy Markdown
Contributor

You guessed it.

@okonomiyaki3000
Copy link
Copy Markdown
Contributor Author

I don't get this error when I run phpunit locally.

@Bakual
Copy link
Copy Markdown
Contributor

Bakual commented Dec 10, 2014

Restarted the jobs to see if it's a temporar fluke.

@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Dec 10, 2014

can we remove the index.html from the cachefolder?

@Hackwar
Copy link
Copy Markdown
Member

Hackwar commented Dec 10, 2014

no, we can not.

@okonomiyaki3000
Copy link
Copy Markdown
Contributor Author

Ah, that's it. But why was that removed...

@okonomiyaki3000
Copy link
Copy Markdown
Contributor Author

ah! I see. I deleted index.html from everywhere then put it back into empty folders. I guess my cache folder wasn't empty because some previous run of phpunit had put stuff in there

@Bakual
Copy link
Copy Markdown
Contributor

Bakual commented Dec 10, 2014

Nice catch Travis 👍

@Hackwar
Copy link
Copy Markdown
Member

Hackwar commented Dec 10, 2014

Travis is happy again. 😄

@wilsonge
Copy link
Copy Markdown
Contributor

Merging as travis is now happy and on review

wilsonge added a commit that referenced this pull request Dec 10, 2014
remove index.html from all folders b... through i...
@wilsonge wilsonge merged commit 2ed7593 into joomla:staging Dec 10, 2014
@Bakual
Copy link
Copy Markdown
Contributor

Bakual commented Dec 10, 2014

Was this your first PR to merge? 😄

@wilsonge
Copy link
Copy Markdown
Contributor

Nope this one :P :P :P 9b46321 then #5086 then this.

@Bakual Bakual added this to the Joomla! 3.4.0 milestone Dec 11, 2014
@okonomiyaki3000 okonomiyaki3000 deleted the kill-indexex-b-i branch December 12, 2014 02:34
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.

6 participants