-
-
Notifications
You must be signed in to change notification settings - Fork 106
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
Incorrect FontAwesome5 Version in docs and update FontAwesome5 to the latest release (5.15.4) #189
Comments
Hi @kumattau thanks for the feedback! I think we forgot to update the Maybe we should update to the latest FontAwesome release: https://github.com/FortAwesome/Font-Awesome/releases/tag/5.15.4 Anyhow, would you like to correct the docs? Let us know! |
OK. I can correct the docs.
I think it is better to updating FontAwesome after implementing reproducible processing. I will try to work in the following order, but it may take some time. Is there a deadline?
|
That sounds like a good plan to me @kumattau thank you so much for all the help! Regarding a deadline I think we will not be doing more releases this year and probably we will check if a new release is worthy by the end of Jaunary but I don't think that we have an actual deadline for the moment 👍 |
I found the compatibility breaking changes between 5.14.0 to 5.15.4. https://github.com/FortAwesome/Font-Awesome/blob/master/UPGRADING.md
FontAwesome5 cannot be updated from a compatibility standpoint, What do you think, @dalthviz ? |
I'm not totally sure @kumattau . What do you think @ccordoba12 ? |
I don't have a problem with that. Those are only two icons and if people require them in their apps, they can simply pin the QtAwesome version they are using. |
Thanks for working on it for me! |
README.md says "FA 5.9.0 features 1,534 free icons in different styles" but FA 5.14.0 seems to be bundled.
I checked the version as follows:
Since the timestamp when the family is changed is embedded, the font checksum changes every time the command is executed, which is to be expected.
The text was updated successfully, but these errors were encountered: