Skip to content

Fix for netcore 2.1 Linq support - #1024

Merged
jeremydmiller merged 1 commit into
JasperFx:masterfrom
jarroda:master
Jun 4, 2018
Merged

jeremydmiller merged 1 commit into
JasperFx:masterfrom
jarroda:master

Conversation

@jarroda

@jarroda jarroda commented Jun 1, 2018 •

Copy link
Copy Markdown
Contributor

This is the best we could come up with for a fix for this. Reflection will have to try to get the overloads for String.Contains at runtime, and what it gets will vary depending on whether its running on 2.0 or 2.1.

@jarroda

jarroda commented Jun 1, 2018

Copy link
Copy Markdown
Contributor Author

Looks like we're waiting on 2.1 support on Appveyor appveyor/ci#2373
Later today perhaps?

@ericgreenmix

Copy link
Copy Markdown
Contributor

@jeremydmiller AppVeyor is now supporting .net 2.1 as of yesterday. Can we get the build re-ran?

@jeremydmiller

Copy link
Copy Markdown
Member

@ericgreenmix @jarroda On its way.

@jeremydmiller
jeremydmiller merged commit ff4c59e into JasperFx:master Jun 4, 2018
lahma pushed a commit to lahma/marten that referenced this pull request Oct 20, 2026
Moves the five JasperFx packages from 2.90.1 to 2.91.0 (released
2026-10-19), spanning 2.90.2. No Marten code change: Marten's
HighWaterDetector already implements FetchCommittedHighWaterCeilingAsync,
which the new high-water stall watchdog (JasperFx#1028) calls, and the
compliance suites are unchanged in this range.

Picked up through the package: subscription agents released on stop
(JasperFx#1022), HighWaterAgent restart fixes (JasperFx#1027/JasperFx#1028),
per-tenant subscription rewind (JasperFx#1025), re-added tenants not held
at their old mark (JasperFx#1026), UTC date text for DateTime members in
Dynamic LINQ criteria (JasperFx#1024), and per-overload EmittedEvents
manifest entries (JasperFx#1021).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017FSnat2TKnusp6MdEZgpHj
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.

3 participants