Skip to content
This repository has been archived by the owner on Jun 5, 2020. It is now read-only.

Use the latest metadata version to fetch icons. #71

Closed
wants to merge 1 commit into from

Conversation

mbovan
Copy link

@mbovan mbovan commented Apr 20, 2018

@tomascordero
Copy link

When will this be merged into master?

@@ -5,7 +5,7 @@ module.exports = function(grunt) {
grunt.initConfig({
download: {
somefile: {
src: ['https://raw.githubusercontent.com/FortAwesome/Font-Awesome/5.0.8/advanced-options/metadata/icons.yml'],
src: ['https://raw.githubusercontent.com/FortAwesome/Font-Awesome/master/advanced-options/metadata/icons.yml'],
Copy link
Owner

@itsjavi itsjavi Jun 24, 2018

Choose a reason for hiding this comment

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

this should always be a tag version. using master is dangerous since it may contain breaking changes, introducing issues in this project.

@itsjavi itsjavi closed this in 1001bcb Nov 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants