Fork me on GitHub
#clojure-dev
<
2022-10-31
>
andy.fingerhut00:10:21

No rush on my part, but if what I have done so far for resolving this issue on transferring the clojure-cheatsheets repo under the Clojure Github organization is not sufficient, happy to try something else: https://github.com/jafingerhut/clojure-cheatsheets/issues/50

Alex Miller (Clojure team)01:10:02

yeah, it's in my list - I think what needs to happen is for you to make someone that is a clojure org owner the owner and then they can transfer it. I am not an owner, but I haven't had a chance to sync up with any of the other folks yet.

Alex Miller (Clojure team)01:10:13

if you want to add Chris Redinger redinger as owner on the repo he's probably the easiest person for me to sync with :)

andy.fingerhut01:10:38

I have just added redinger as a collaborator on the project. I do not know if that is the same as an owner.

andy.fingerhut01:10:14

Ah, I just found the option to transfer the ownership of the repo to another user id. I attempted to transfer it to redinger, and it says it is in the process of being transferred

seancorfield02:10:45

Cool to see the cheatsheets getting integrated into http://clojure.org! Thank you!

andy.fingerhut19:11:29

Looks like the Github repo is transferred to clojure Github org now. Cool!

andy.fingerhut19:11:47

I suspect that if others want to update and regenerate the cheatsheet in the future, you might prefer different ways to automate it than are currently checked in. I'm happy to answer questions about anything there that is confusing, or walk someone through my steps and make sure they are all written in some README file in that repo somewhere.

Alex Miller (Clojure team)19:11:34

Yeah, I will throw it in a GitHub action probably

Alex Miller (Clojure team)19:11:54

I started working on recreating parts of the gh pages from your home repo too

👍 1
borkdude12:10:10

Found that *print-readably* doesn't have :dynamic set on the metadata:

user=> (:dynamic (meta #'clojure.core/*print-readably*))
nil
Want a ticket/patch for that?

borkdude12:10:03

Same for flush on newline and print-dup

Alex Miller (Clojure team)13:10:34

those are declared as dynamic

borkdude13:10:57

I know, but the metadata doesn't indicate this

borkdude13:10:55

I ran into this when copying vars into a SCI context based on the metadata (macro true, dynamic true)

Alex Miller (Clojure team)13:10:47

might be different reasons for some of these, like some might be due to deficiencies in defonce (which I think has a ticket)

Alex Miller (Clojure team)13:10:07

but I'd also want to check if this is really the issue with defonce reevaluation nuking meta

borkdude13:10:03

This works:

clj -Sdeps '{:deps {org.clojure/clojure {:local/root "target/clojure-1.12.0-master-SNAPSHOT.jar"}}}'

Clojure 1.12.0-master-SNAPSHOT
user=> (:dynamic (meta #'*print-dup*))
true

borkdude13:10:21

(with the extra key in place in core.clj)

borkdude13:10:32

I created the issue here: https://clojure.atlassian.net/browse/CLJ-2734 Let me know if a patch is welcome

borkdude13:10:36

Patch attached: https://clojure.atlassian.net/browse/CLJ-2734 Assigned to the first of three Alex Millers that popped up :)

andy.fingerhut13:10:57

I always knew Alex must have clones 🙂

🤯 2
Alex Miller (Clojure team)14:10:28

the jira transition to Cloud was ... complicated :) resulted in a bit of a clone army which I've given up fixing

stormtrooper 4
andy.fingerhut14:10:39

clone army: bug -> feature

😄 1