change the YAML #1149

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@timini

I think this is the correct convention that's used in core.

Tim Richardson Update YAML
I think this is the correct convention that's used in core.
8421dc9
@arian arian commented on the diff Aug 29, 2012
Source/Element/Elements.From.js
@@ -12,10 +12,7 @@ license: MIT-style license
authors:
- Aaron Newton
-requires:
- - Core/String
- - Core/Element
- - /MooTools.More
+requires: [String, Element, More]
@arian
arian Aug 29, 2012

Doesn't really matter, both are valid. Because String and Element are not in More, it has to be Core/String to indicate that it's in the Core package.
Also More/More.js provides MooTools.More (https://github.com/mootools/mootools-more/blob/master/Source/More/More.js#L25) so that should stay MooTools.More.

@SergioCrisostomo
MooTools member

@timini you are right that Core and More use different YAML headers. The packager does accept both cases, still we will make a YAML cleanup next week preparing for the 1.5 release.

Thank you for your input on this!

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