Skip to content

remove index.html from all folders starting with l#5381

Merged
Bakual merged 1 commit intojoomla:stagingfrom
okonomiyaki3000:kill-indexes-l
Dec 10, 2014
Merged

remove index.html from all folders starting with l#5381
Bakual merged 1 commit intojoomla:stagingfrom
okonomiyaki3000:kill-indexes-l

Conversation

@okonomiyaki3000
Copy link
Copy Markdown
Contributor

yep.

@Bakual
Copy link
Copy Markdown
Contributor

Bakual commented Dec 10, 2014

I'm fine with removing the index files from our own libraries.
@nikosdion Are you going to remove index.html from fof? If so that could be done already, if not they should stay
@mbabker Do you remember if we added the index.html files to the other libraries (like phpass, idn stuff) ourself?

@Hackwar
Copy link
Copy Markdown
Member

Hackwar commented Dec 10, 2014

We added them all ourselfs. I would simply remove them. In worst case, they would be added back in with the next release of an extension, but I doubt that.

@okonomiyaki3000
Copy link
Copy Markdown
Contributor Author

CodeMirror normally has a bunch of html files that are meant to show demos and stuff. I stripped them out in the latest update.

@nikosdion
Copy link
Copy Markdown
Contributor

@Bakual You can remove them from the Joomla! staging branch and I'll also remove them from the FOF repository.

Bakual pushed a commit that referenced this pull request Dec 10, 2014
remove index.html from all folders starting with l
@Bakual Bakual merged commit 315d234 into joomla:staging Dec 10, 2014
@Bakual
Copy link
Copy Markdown
Contributor

Bakual commented Dec 10, 2014

Thanks!

@Bakual Bakual added this to the Joomla! 3.4.0 milestone Dec 11, 2014
@okonomiyaki3000 okonomiyaki3000 deleted the kill-indexes-l branch December 12, 2014 02:33
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