Skip to content

Update tests and CI#4

Open
aminvielle wants to merge 7 commits intomasterfrom
update-tests-and-ci
Open

Update tests and CI#4
aminvielle wants to merge 7 commits intomasterfrom
update-tests-and-ci

Conversation

@aminvielle
Copy link
Copy Markdown
Collaborator

Commit dc40a49 disabled many tests so that a new version of flexmark could be quickly released.
This PR reverts dc40a49 and :

  • Fix some broken tests and disable much less precious unit tests
  • Add a minimalistic Github CI

aminvielle and others added 5 commits April 9, 2026 16:46
Set default locale to ENGLISH before tests to ensure consistent
decimal separator in floating-point output regardless of system locale.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Mark "Typographic quotes interaction: 1" as IGNORE in the spec file.
The test is system-dependent: abbreviation matching for accented
characters (e.g. É.U.) behaves inconsistently across locales due
to regex word boundary handling with non-ASCII characters.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Parboiled (used internally by pegdown) relies on reflective access
to JVM internals that are restricted since Java 9, causing a runtime
error when creating extended parser classes. Skip the tests when not
running on Java 8.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Runs mvn test on every push and pull request targeting master.
Uses Java 21 (Temurin) with Maven dependency caching.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@aminvielle aminvielle force-pushed the update-tests-and-ci branch from 0cc6dad to e4630b7 Compare April 13, 2026 15:00
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.

1 participant