Skip to content

Conversation

ikappaki
Copy link
Contributor

@ikappaki ikappaki commented Jun 1, 2025

Hi,

could please consider patch to add the goog.string/htmlEscape fn. It addresses #105.

There doesn’t appear to be a test suite available, so I tested this manually by inserting the following into resources/public/index.html and observing the console output:

      (require '[goog.string :as gstring])
      (println :escape (gstring/htmlEscape "<hello>">))
=> :escape &lt;hello&gt;

Thanks

Please answer the following questions and leave the below in as part of your PR.

  • I have updated the CHANGELOG.md file with a description of the addressed issue.

@borkdude borkdude merged commit 07c8d48 into babashka:main Jun 2, 2025
@borkdude
Copy link
Collaborator

borkdude commented Jun 2, 2025

Thanks!

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.

2 participants