Skip to content
This repository was archived by the owner on Apr 14, 2021. It is now read-only.

GH95 Trouble building #101

Merged
merged 1 commit into from
Jun 1, 2015
Merged

GH95 Trouble building #101

merged 1 commit into from
Jun 1, 2015

Conversation

kate2753
Copy link
Collaborator

Adding missing src\cldr file to resolve build issues. I've copied everything prefixed with $$cldr$$ from dist/Intl.js file.

grunt build is now successful:

$ grunt build
Running "bundle_jsnext:dest" (bundle_jsnext) task
INFO: Using a global wrapper module for "src/main.js", exporting namespace "IntlPolyfill"
>> Bundled library written in dist/Intl.js

Running "uglify:build" (uglify) task

Running "cjs_jsnext:dest" (cjs_jsnext) task
>> CommonJS library written in tmp/

Running "copy:src" (copy) task
Copied 4 files

Running "concat:complete" (concat) task
File dist/Intl.complete.js created.

Done, without errors.

All unit tests pass with dist files generated by grunt build.

andyearnshaw added a commit that referenced this pull request Jun 1, 2015
@andyearnshaw
Merge pull request #101 from kate2753/gh95
c8d1989 
GH95 Trouble building
@andyearnshaw andyearnshaw merged commit c8d1989 into andyearnshaw:master Jun 1, 2015
@andyearnshaw andyearnshaw mentioned this pull request Jun 1, 2015
Closed
@caridy
Copy link
Collaborator

caridy commented Jun 8, 2015

ah... my bad. I will diff it with my local copy, and we can go from there... but it looks good at first glance @kate2753

@caridy caridy mentioned this pull request Jun 8, 2015
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kate2753 @caridy @andyearnshaw