lua errors when trying to render revealjs with fontawesome extension about fontawesome HOT 22 CLOSED

@eitsupi No, there will be a 1.0. I just don't want to encourage people to install an old version of quarto.

from fontawesome.

@eitsupi That's a great idea.

from fontawesome.

@morrisLuke The extension should not be asking users to use the pre-release / not stable version without a working solution with the stable version.

from fontawesome.

@morrisLuke Glad you got it sorted out, but no need to apologize, this is clearly a bug on the advertised version requirement.

... Oh, I see the problem now, 🤦 . The issue is that 1.1 does not have the code that checks for version requirements. That's annoying. At least we are soon going to have a stable 1.2 version, and that should solve this problem.

from fontawesome.

He updated Quarto to v1.2 pre-release: https://quarto.org/docs/download/

from fontawesome.

@cscheid Now that quarto 1.2 has been released, how about creating the 1.0.0 release of this repo?

from fontawesome.

P.S. new to issue reporting, so let me know if anything else is needed

from fontawesome.

Are you running a v1.2 prerelease version of quarto?

from fontawesome.

This is related to #19 and the issue comes from the upgrade performed based on Quarto 1.2 code change.

The solution would be to add tags/releases for the following commits:

• 63f2000 (latest version compatible with Quarto v1.1)
• HEAD (6206fc6) (first version compatible with Quarto v1.2)

With tags, it would be possible to use:

quarto install extension quarto-ext/[email protected]

@cscheid Should the extensions in https://github.com/quarto-ext be following Quarto version, at least for Major and Minor, then the patch part can be used for fixes?

from fontawesome.

ahhh.... Looks like I've got 1.1.251, the stable on quarto's get started page.

from fontawesome.

It's fine, but the extension is currently requiring the pre-release, but the latest tag is way too old to be used.

from fontawesome.

Just updated to the 1.2.231 prerelease and things are resolved. Thanks for tolerating a noob

from fontawesome.

@mcanouil No, what should have happened is that we should update the quarto version requirement for this package.

from fontawesome.

Meanwhile, tag/release can help @cscheid 😉

from fontawesome.

@mcanouil no, it cannot, not in this case! the issue is with quarto's version. Quarto 1.1 lacks the code that checks extension versions.

Releases will not help here because we are not ever going to have the HEAD of this repo expecting an old version of quarto. The right solution would be to backport the version checking code in 1.2 back to 1.1. But that doesn't solve the issue that the current installed base of RStudio etc still will have the old, non-checking code. And the new releases of RStudio will be getting a v1.2 release anyway, like I mentioned.

from fontawesome.

The idea is to tag/release 63f2000 with for example vO.O.2
Then one can quarto install extension quarto-ext/[email protected] if using Quarto < v1.2

from fontawesome.

The extension support in 1.1 is poor enough that I truly would rather this not be easy to do. v1.2 is going to be out in a couple of weeks. I'm sorry, but this discussion is not particularly productive at this point.

from fontawesome.

It's fine 😉
I thought that the idea to allow to target tags when installing/using extensions/formats was for these situations among others. I am most likely missing some "long run" infos. 🤷‍♂️

from fontawesome.

Just updated to the 1.2.231 prerelease and things are resolved. Thanks for tolerating a noob

Please, share the code you used to resolve this issue.

from fontawesome.

Hmmm, does this mean that there are no plans to release version 1.0.0?
(I understand that quarto 1.1 does not check the required version, but isn't that unrelated to not creating a release or tag?)

from fontawesome.

from fontawesome.

He updated Quarto to v1.2 pre-release: quarto.org/docs/download

Thank you, updating Quarto to the latest version (pre-release in this case) helped to solve the error related to the fontawesome.

from fontawesome.