Skip to content
This repository has been archived by the owner on May 1, 2019. It is now read-only.

Enhancement: Removed asset manager #433

Merged
merged 6 commits into from
Feb 27, 2015

Conversation

ins0
Copy link
Contributor

@ins0 ins0 commented Feb 27, 2015

This removes the asset manager. In the current state the asset manager don't provides real effort nor we use multi module public folders.

@localheinz
Copy link
Member

👍

@@ -10,7 +10,6 @@
"ezyang/htmlpurifier": "4.6.*",
"monolog/monolog": "~1.12",
"php": "~5.5",
"rwoverdijk/assetmanager": "1.3.*",
Copy link
Member

Choose a reason for hiding this comment

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

@ins0

You also need to run

$ composer update rwoverdijk/assetmanager

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, a question - what is the point that the composer.lock file is in the version control?

Copy link
Member

Choose a reason for hiding this comment

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

@ins0 the composer.lock MUST be under version control for deployments

@@ -1,4 +1 @@
img
css
js
favicon.ico
Copy link
Member

Choose a reason for hiding this comment

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

@ins0

I think this .gitignore can be removed, can't it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah

@@ -1,4 +0,0 @@
img
Copy link
Member

Choose a reason for hiding this comment

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

Can we actually use bower for this stuff?

Copy link
Member

Choose a reason for hiding this comment

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

@Ocramius

👍, but out of scope of this PR, I think!

Copy link
Member

Choose a reason for hiding this comment

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

We are replacing with directly committed assets, heh

Copy link
Member

Choose a reason for hiding this comment

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

Nvm, we are just moving the assets. But yes, we should use bower here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

@Ocramius Ocramius self-assigned this Feb 27, 2015
Ocramius added a commit that referenced this pull request Feb 27, 2015
@Ocramius Ocramius merged commit 15ac746 into zendframework:master Feb 27, 2015
@Ocramius
Copy link
Member

🚢 :shipit:

@ins0 ins0 deleted the enhancement/asset-manager branch February 27, 2015 20:31
@ins0
Copy link
Contributor Author

ins0 commented Feb 27, 2015

thanks @localheinz and @Ocramius

@gianarb
Copy link
Contributor

gianarb commented Feb 27, 2015

Thanks!!!! @ins0 👍 👍

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