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

[Bug] The package for Meteor is outdated at 4.2.0_2 #5533

Closed
msamoylov opened this issue Jan 30, 2015 · 18 comments
Closed

[Bug] The package for Meteor is outdated at 4.2.0_2 #5533

msamoylov opened this issue Jan 30, 2015 · 18 comments

Comments

@msamoylov
Copy link

It's not possible to upgrade to 4.3.0 from https://atmospherejs.com/fortawesome/fontawesome

@tagliala
Copy link
Member

#5062

@dandv
Copy link

dandv commented Jan 30, 2015

@splendido is on it!

@splendido
Copy link

Ok, I''l try to see what I can do.

@tagliala would you accept a PR which adds only a package.js file to the other PM files already present?

In case you'll feel comfortable with this, the only other changes we'd need would be to make the version field appearing inside the package.js files updated when you make new builds.
In case, could you give some pointer about your building process? I'm not seeing grunt or gulp files...

Tnx,
Luca

Update: it wouldn't be bad to have also a tests.js file specifically for meteor tests to be put where you prefer.

@tagliala
Copy link
Member

@splendido
I would like to include official meteor support in 5.0.0 version, but these decisions are up to Dave.

Please send your PR against the 5.0.0-wip branch

In case, could you give some pointer about your building process? I'm not seeing grunt or gulp files...

It's still jekyll based. I mean that jekyll is also responsible of creating .less and .srcc dist files. Very ugly but it does the job. There is a proposal to change the build process ( #1166 ) but this is entirely up to Dave

So: bundle install and jekyll build should be enough: https://github.com/FortAwesome/Font-Awesome#hacking-on-font-awesome

Update: it wouldn't be bad to have also a tests.js file specifically for meteor tests to be put where you prefer.

Put it where you think it is best

@splendido
Copy link

Thanks for your availability @tagliala!

Just to confirm: PM files are updated by hands outside of the jekyll build right?
See latest version updates
package.js will be also on the list...

@tagliala
Copy link
Member

hmmm not all of them. I don't know if we need to add others to the list (package.js, components.js). I'm doing that manually at the moment

@splendido
Copy link

I'd say using gulp to automatize the build process would be the best option.
It's getting more and more reliable and it's actually taking over on grunt!

...to tell you just one example, Jack from SemanticUI is heavily using it (he's refactoring the whole building process right now actually...)

Ideally, all PM files should be automatically generated from the build process picking up the latest package's information from the package.json, so that you won't have to modify them by hands anymore.

But this choice is up to you, surely not up to me. Just my two cents.

Btw, I'll prepare another (leaner) proposal for the meteor integration by means of a PR on the 5.0.0-wip branch.

Tnx!

@tagliala
Copy link
Member

But this choice is up to you, surely not up to me. Just my two cents.

I want a better build process, the choice is up to Dave. Don't know his priorities for 5.0.0 :)

@splendido
Copy link

Yeah, sorry, with you I meant you package maintainers. Dave is obviously involved ;-)

@splendido
Copy link

Done, See #5543!

@serkandurusoy
Copy link

Hi there, the package on Meteor is still 4.2.0_2 :(

@splendido
Copy link

We're still waiting for #5543 to be merged to get automatic updates...
Before that @dandv can manually update it.

@ErikAbele
Copy link

Would be really nice to get this fixed now and for all times… TIA!

@splendido
Copy link

@dandv could you make another version bump by hands?

@ErikAbele
Copy link

Ping?

@dandv
Copy link

dandv commented Mar 29, 2015

Fixed!

@splendido
Copy link

now at version 4.4.0: see it on atmoshpere
I guess this issue can be closed now...

@tagliala
Copy link
Member

@splendido thanks!

closing here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants