cryogen

Jakub Holý (HolyJak) 2020-10-16T07:14:21.000600Z

set the channel topic: http://cryogenweb.org/ - static sites by @carmen & Co.

❤️ 1
Jakub Holý (HolyJak) 2020-10-16T07:14:40.001Z

🙏 https://github.com/cryogen-project/cryogen-asciidoc/pull/8

👍 2
Jakub Holý (HolyJak) 2020-10-16T12:15:46.002400Z

☝️ has been extended to add support for setting asciidoctor options and attributes in config.edn. Please ping me when it is out so that I can use it on my blog 🙏 @carmen / @yogthos

👍 1
Eugen 2020-10-16T12:44:59.003200Z

Nice. Looking forward to seeing some improvements in cryogen web.

Eugen 2020-10-16T12:48:28.004500Z

Once I get back to my website I hope to contribute some features

❤️ 1
Eugen 2020-10-16T12:54:50.008700Z

My focus is on asciidoc as I believe it has more potential (although less tooling suport atm). I also the conventions and structure that https://antora.org/ has around building (documentation) websites from different git repos and modules.

Eugen 2020-10-16T12:57:33.009800Z

They also have a nice solution for providing an UI

yogthos 2020-10-16T13:07:42.010500Z

thanks for the PR, just pushed out 0.3.0 version of cryogen-asciidoc with the update 🎉

Jakub Holý (HolyJak) 2020-10-16T18:05:35.010900Z

@yogthos merging the multiple extensions PR broke asciidoc b/c it depends on https://github.com/cryogen-project/cryogen-core/pull/146 being merged into core and then depending on that version > Syntax error (IllegalArgumentException) compiling reify* at (cryogen_asciidoc/core.clj:22:3). > Can't define method not in interfaces: exts Here is a PR to revert it https://github.com/cryogen-project/cryogen-asciidoc/pull/10

Jakub Holý (HolyJak) 2020-10-16T18:41:59.011400Z

Thanks for the merge! Can you re-release 0.3.0 to clojars? or bump the version and release? 🙏

yogthos 2020-10-16T18:42:40.011600Z

oh yeah I'll push a new version out in a few minutes

yogthos 2020-10-16T18:55:19.011800Z

ok everything should be up on Clojars

Jakub Holý (HolyJak) 2020-10-16T19:08:05.012Z

❤️

yogthos 2020-10-16T19:29:38.012200Z

thanks again for doing the PRs 🙂

dorab 2020-10-16T23:01:20.012400Z

• Sorry for the need to simultaneously update the various cryogen repos to handle the multiple-extensions PR. The list of PRs that need to be updated together are:`cryogen-asciidoc` PR number 7 • cryogen-docs PR number 32 • cryogen-markdown PR number 13 • cryogen-flexmark PR number 6 and cryogen-core PR number 146

dorab 2020-10-16T23:15:46.012700Z

As @holyjak noted, the dependencies need to be updated after the PRs are applied. I did not want to put the dependency changes into the PRs since I didn't know what the new version numbers would be. I think the current states of the sub-projects of cryogen-project are inconsistent. What is needed is to apply all the PRs in each of the sub-projects mentioned above and then to update all the dependencies.

dorab 2020-10-16T23:21:04.012900Z

Please let me know how I can help.

dorab 2020-10-17T00:12:22.013100Z

@yogthos Just to be clear, ALL the related PRs need to be applied and then all the dependencies updated. It is not enough to apply just some of them. This is because the name of the protocol method is changed from ext to exts. Happy to do this if I have write access to the repos.

yogthos 2020-10-17T00:14:10.013300Z

I merged all the PRs I think, but check with @carmen about access

carmen 2020-10-17T00:39:33.013500Z

@dorab I sent you an invite to the org!

dorab 2020-10-17T00:50:00.013700Z

Thanks. @carmen. Joined. @yogthos thanks. I'll take a look and make the required changes.

yogthos 2020-10-17T01:01:22.013900Z

🎉

dorab 2020-10-17T01:01:56.014100Z

Does someone want to review the PRs before I merge them? I feel a bit strange merging my own PRs. But happy to do so if needed.

dorab 2020-10-17T01:18:29.014300Z

I'm going ahead with merging the PRs.

yogthos 2020-10-17T01:25:15.014500Z

Should I push out another release for core and asciidoc?

dorab 2020-10-17T01:48:26.014700Z

I have applied the changes to all the relevant repos. BUT, I have not yet updated any of the version numbers. I will update the "patchlevel" versions of all the relevant repos and change the dependencies. Then the repos should be ready to be released.

dorab 2020-10-17T01:59:51.014900Z

@yogthos, is the current cryogen-core master branch the same as 0.3.2? Asking because the last tag on the repo is 0.3.1 but the README says 0.3.2.

dorab 2020-10-17T02:41:13.015100Z

I downloaded and looked at 0.3.2 and it looks like it matches master.

dorab 2020-10-17T03:00:18.015300Z

I updated the master branch of the following repos:

dorab 2020-10-17T03:01:22.015500Z

cryogen-asciidoc to 0.3.2

dorab 2020-10-17T03:02:39.015700Z

cryogen-markdown to 0.1.13

dorab 2020-10-17T03:03:16.015900Z

cryogen-flexmark to 0.1.4

dorab 2020-10-17T03:03:41.016100Z

cryogen-docs to 0.1.1

dorab 2020-10-17T03:04:36.016300Z

I updated the project.clj in each of these repos and created a tagged release in each repo.

dorab 2020-10-17T03:05:15.016500Z

I'll leave the publishing of each of these repos to clojars to @yogthos.

dorab 2020-10-17T03:07:48.016800Z

I'll note that the circleci build of cryogen-docs failed . Presumably because the correct versions of the dependencies are not yet on clojars. Other circleci jobs might fail too, for the same reason.

dorab 2020-10-17T03:10:14.017Z

After publishing to clojars, it might be good to run the tests in cryogen-core with all the latest versions.

dorab 2020-10-17T03:11:15.017200Z

Going to bed now. Will check back here tomorrow morning.

dorab 2020-10-17T03:15:41.017400Z

One other thing ... I did not make any changes to cryogen-core since 0.3.2 seemed to have all the necessary changes.

yogthos 2020-10-17T03:46:13.017600Z

fantastic, just released all the updated libraries

yogthos 2020-10-17T03:47:11.017800Z

and @carmen can take a look at the docs

yogthos 2020-10-17T03:48:21.018Z

oh it looks like the docs wasn't building because I haven't pushed the new libraries to clojars yet

Jakub Holý (HolyJak) 2020-10-17T14:28:21.018200Z

@dorab The advice from the Clojure philosophy would be not to change the protocol (a breaking change) but to create a new one or one in a new namespace. Then you could simply add it to core, release it, and the plugins could choose to extend it in their own time

dorab 2020-10-17T17:14:48.018400Z

Thanks. I agree. I did think about just adding an exts method to the existing Markup protocol. If I did that, it is my understanding that since the protocol itself has changed, that would still require synchronized updates. Meaning that if someone updated cryogen-core to the version containing the new Markup protocol, they could not continue to use the old implementations of the previous version of the Markup protocol. Is my understanding correct?

✅ 1