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

renderUnversioned = false still renders laika/versionInfo.json #265

Closed
armanbilge opened this issue Mar 7, 2022 · 6 comments · Fixed by #266
Closed

renderUnversioned = false still renders laika/versionInfo.json #265

armanbilge opened this issue Mar 7, 2022 · 6 comments · Fixed by #266
Labels
Milestone

Comments

@armanbilge
Copy link
Member

To reproduce:

  1. checkout https://github.com/http4s/http4s-jdk-http-client/tree/series/0.7
  2. sbt docs/tlSite
  3. find site/target/docs/site/laika/versionInfo.json

My understanding from typelevel/sbt-typelevel#100 (comment) is that this should not happen.

In case of using the renderUnversioned flag in the intended way, versionInfo.json will also only ever be published by the main branch (as it is an unversioned document itself).

None of the other unversioned documents are rendered, so I assume that it's configured correctly.

Thanks!

@jenshalm
Copy link
Contributor

jenshalm commented Mar 7, 2022

I need to check this, it might be that this is an oversight where I temporarily included it to test the version dropdown on output with that config. But I really wonder how it's even able to slip through... :-)

@armanbilge
Copy link
Member Author

No worries! :)

I noticed you added this to the 0.19.0 milestone. A patch release of 0.18 would be ideal, but if not possible can you propose any workaround?

@jenshalm
Copy link
Contributor

jenshalm commented Mar 7, 2022

There is no milestone at the moment for an 0.18.2 release, that doesn't mean I cannot do one if required. In the past I rarely did patch releases simply because there did not seem to be any demand for it. Regarding potential workarounds I need to first check how/why it's actually slipping through.

@armanbilge
Copy link
Member Author

Thanks, whenever you have a chance to look into it!

In the past I rarely did patch releases simply because there did not seem to be any demand for it.

Yes, sorry for all the new demands 😬 Laika's getting to be a big hit in Typelevel :) whenever possible, anything that can be fixed with a patch works really great for us, otherwise I can try to absorb any workarounds into sbt-typelevel.

On that note, if there's anything I can do to improve your build/ci infrastructure to make it easier to release patches, please please let me know! :)

@jenshalm
Copy link
Contributor

jenshalm commented Mar 7, 2022

Laika's getting to be a big hit in Typelevel :)

Oh is it? Despite the fact that there are voices saying "it is not the prettiest"? (My eyes are everywhere... 😄 )

On that note, if there's anything I can do to improve your build/ci infrastructure to make it easier to release patches, please please let me know! :)

I'm still releasing like a monkey. 😄 And I might want to keep it that way as Laika produces so many visual artefacts, some of which are notoriously difficult to test in an automated way (e.g. PDF output). Which is why I always stop half-way through a release and check various outputs, PDF, EPUB and Laika's rendered manual.

@armanbilge
Copy link
Member Author

armanbilge commented Mar 7, 2022

oh my 🙈 😳 well then you also know I plan to move the Cats website onto Laika, and if that's not a feather in your cap then I don't know what is! 😁

Which is why I always stop half-way through a release and check various outputs, PDF, EPUB and Laika's rendered manual.

I can fix this! Here's how:

If we set you up with CI release, the release doesn't go out until you push a tag. And at that point, it's re-running CI on that commit (except this time publishing), since CI already ran on that commit when you first pushed it to the branch. So, the results should be identical.

So, that means you just need to be convinced of the worthiness of that commit. During CI, we can upload the visual artifacts as GH artifacts which you can directly download and verify from the GH CI summary. Then, if you are satisfied, you can create the tag.

What do you think, would that work for you?

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

Successfully merging a pull request may close this issue.

2 participants