Skip to content

fix: update OCL to v2025.11.2#353

Merged
targos merged 4 commits intomainfrom
2025.11.2
Nov 25, 2025
Merged

fix: update OCL to v2025.11.2#353
targos merged 4 commits intomainfrom
2025.11.2

Conversation

@targos
Copy link
Member

@targos targos commented Nov 19, 2025

No description provided.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 19, 2025

Deploying openchemlib-js with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2de9b27
Status: ✅  Deploy successful!
Preview URL: https://3de6af27.openchemlib-js.pages.dev
Branch Preview URL: https://2025-11-2.openchemlib-js.pages.dev

View logs

@targos
Copy link
Member Author

targos commented Nov 19, 2025

@lpatiny The issue is fixed but this test doesn't fail:

// this test could fail once this bug is fixed: https://github.com/Actelion/openchemlib/issues/158
expect(rxn2).toMatchSnapshot();

@targos
Copy link
Member Author

targos commented Nov 25, 2025

@lpatiny All these new tests also pass on the main branch so the OCL fix is still not tested (or not working)

@lpatiny
Copy link
Member

lpatiny commented Nov 25, 2025

This should be the right test

@targos
Copy link
Member Author

targos commented Nov 25, 2025

Yes I confirm, thanks!

@targos targos merged commit c429f45 into main Nov 25, 2025
6 checks passed
@targos targos deleted the 2025.11.2 branch November 25, 2025 09:16
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