Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Address JDK 19 Thread/sleep Clojure interop #504

Merged
merged 2 commits into from
Nov 30, 2022

Conversation

lread
Copy link
Collaborator

@lread lread commented Nov 29, 2022

JDK 19's Thread/sleep now also accepts a Duration. Add a ^long type hint to remove this new ambiguity for Clojure interop.

Closes #503

Please complete and include the following checklist:

  • I have read CONTRIBUTING and the Etaoin Developer Guide.

  • This PR corresponds to an issue that the Etaoin maintainers have agreed to address.

  • This PR contains test(s) to protect against future regressions

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

JDK 19's Thread/sleep now also accepts a Duration.
Add a ^long type hint to remove this new ambiguity for Clojure interop.

Closes clj-commons#503
@lread
Copy link
Collaborator Author

lread commented Nov 30, 2022

CI failures are #447, will address separately.

@lread lread merged commit cdda5e4 into clj-commons:master Nov 30, 2022
ageneau pushed a commit to ageneau/etaoin that referenced this pull request Aug 10, 2023
* Address JDK 19 Thread/sleep Clojure interop

JDK 19's Thread/sleep now also accepts a Duration.
Add a ^long type hint to remove this new ambiguity for Clojure interop.

Closes clj-commons#503

* docs: update change log
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.

Execution error (IllegalArgumentException)
1 participant