GitHub user 1ambda opened a pull request:

    https://github.com/apache/zeppelin/pull/1875

    [ZEPPELIN-1926] Remove grunt googlefonts task

    ### What is this PR for?
    
    There is no reason to download fonts files whenever we build.
    
    - Google font doesn't change frequently.
    - We can use google fonts in open source 
(https://developers.google.com/fonts/faq)
    
    > Yes. The open source fonts in the Google Fonts catalog are published 
under licenses that allow you to use them on any website, whether it’s 
commercial or personal. 
    
    
    ### What type of PR is it?
    [Improvement]
    
    ### Todos
    
    Fixed at once
    
    ### What is the Jira issue?
    
    [ZEPPELIN-1926](https://issues.apache.org/jira/browse/ZEPPELIN-1926)
    
    ### How should this be tested?
    
    1. Run zeppelin-web dev mode
    2. Open browser and check 404
    
    ### Screenshots (if appropriate)
    
    None
    
    ### Questions:
    * Does the licenses files need update? - NO
    * Is there breaking changes for older versions? - NO
    * Does this needs documentation? - NO


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/1ambda/zeppelin 
ZEPPELIN-1926/remove-grunt-googlefonts-task

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zeppelin/pull/1875.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1875
    
----
commit c752edb31d1fa7a14a7b7d5988637852336ef382
Author: 1ambda <1am...@gmail.com>
Date:   2017-01-09T00:10:33Z

    fix: Remove grunt googlefonts

commit 27d6c7fef6a32ad4d3316f8070b8e0f27573908a
Author: 1ambda <1am...@gmail.com>
Date:   2017-01-09T00:10:42Z

    chore: Add fonts files

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to