Skip to content

Conversation

@adityaathalye
Copy link
Member

@adityaathalye adityaathalye commented Feb 28, 2024

I had been meaning to make this simplification for a good while.

Thanks for pushing these changes, @kirankulkarni @0niket and @kapilreddy . I randomly noticed them today and have taken the liberty to create a PR, to keep the change observable from the in-clojure repo. Will review and merge at some point.

@ghoseb ghoseb merged commit 2d2be66 into inclojure-org:master Mar 4, 2024
@adityaathalye
Copy link
Member Author

@ghoseb Please be advised, the PR required fixes and cleanups. I will push those changes separately.

@ghoseb
Copy link

ghoseb commented Mar 4, 2024

@adityaathalye Sorry about that. I thought it was ready for merging. I looked at all the changes, I didn't see anything amiss. Apologies.

@adityaathalye
Copy link
Member Author

No probs, they are small inconsistencies, easy to miss in the less than adequate Github PR review interface. I checked the diff locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants