Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added a note about the total deprecation of YUI #4195

Merged
merged 2 commits into from
Oct 1, 2014
Merged

Added a note about the total deprecation of YUI #4195

merged 2 commits into from
Oct 1, 2014

Conversation

javiereguiluz
Copy link
Member

Q A
Doc fix? no
New docs? yes
Applies to 2.3+
Fixed tickets -

Maybe we could also close the related issue #1841?

worry! See :doc:`/cookbook/assetic/uglifyjs` for an alternative.
The YUI Compressor is `no longer maintained by Yahoo`_ but by an independent
volunteer. Moreover, Yahoo has decided to `stop all new development on YUI`_
and to move to other modern alternatives such as NodeJS.
Copy link
Member

Choose a reason for hiding this comment

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

Afaik, the official name is Node.js.

Copy link
Member Author

Choose a reason for hiding this comment

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

I always have the same doubt. In their homepage they use at least four variants: "node.js", "Node", "Node.js" and "Node.js®"

Copy link
Member

Choose a reason for hiding this comment

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

Yes, but Node.js is the one used in their trademark policy.

Copy link
Member Author

Choose a reason for hiding this comment

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

@xabbuh enough said! That's definitely the proper name. I've just fixed it. Thanks for reporting this error.

@wouterj
Copy link
Member

wouterj commented Sep 8, 2014

👍

(can someone recall why we still have this chapter in the docs?)

@weaverryan
Copy link
Member

@javiereguiluz can you re-push this branch? It's missing from your fork now :).

Yea, we don't need this document anymore, but with this clear warning on the top, I think it still serves a purpose for blog posts, stackoverflow, etc that are linking to this post. Instead of a 404, they'll (ideally) see this big warning, then go check out uglify.

@javiereguiluz
Copy link
Member Author

I'm afraid that we have the same problem as the other issues: I deleted my fork before all the pending PR were merged. I'm really sorry for all the troubles caused :(

@weaverryan
Copy link
Member

No worries Javier! I was using an outdated version of the merge tool - I was able to merge this no problem :). Thanks!

@weaverryan weaverryan merged commit f5021d5 into symfony:2.3 Oct 1, 2014
weaverryan added a commit that referenced this pull request Oct 1, 2014
…eguiluz)

This PR was merged into the 2.3 branch.

Discussion
----------

Added a note about the total deprecation of YUI

| Q             | A
| ------------- | ---
| Doc fix?      | no
| New docs?     | yes
| Applies to    | 2.3+
| Fixed tickets | -

Maybe we could also close the related issue #1841?

Commits
-------

f5021d5 Fixed the official name of Node.js
d027c80 Added a note about the total deprecation of YUI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants