Skip to content

Comments

Add goog.string/htmlEscape#106

Merged
borkdude merged 1 commit intobabashka:mainfrom
ikappaki:feat/html-escape
Jun 2, 2025
Merged

Add goog.string/htmlEscape#106
borkdude merged 1 commit intobabashka:mainfrom
ikappaki:feat/html-escape

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