Skip to content

Include grouped alert query results inline#71

Merged
nblumhardt merged 6 commits into
datalust:masterfrom
nblumhardt:results-inline
Jun 4, 2026
Merged

Include grouped alert query results inline#71
nblumhardt merged 6 commits into
datalust:masterfrom
nblumhardt:results-inline

Conversation

@nblumhardt
Copy link
Copy Markdown
Member

@nblumhardt nblumhardt commented Jun 4, 2026

image

Fixes #70

@nblumhardt nblumhardt marked this pull request as ready for review June 4, 2026 03:34
@nblumhardt
Copy link
Copy Markdown
Member Author

Ignore whitespace when diffing.

.NET 6 is no longer present on build workers; moved to .NET 8 to get the app onto a more modern API surface (Seq 2024+, which covers just about everyone in production usage at this point). Solution and SDK are on .NET 10.

<PropertyGroup>
<TargetFramework>netstandard2.0</TargetFramework>
<VersionPrefix>1.0.0</VersionPrefix>
<TargetFramework>net8.0</TargetFramework>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason not to target net10 here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only 2026.1 supports .NET 10, we'd break most consumers of the package.

@nblumhardt nblumhardt merged commit 248f39e into datalust:master Jun 4, 2026
1 check passed
@nblumhardt nblumhardt mentioned this pull request Jun 7, 2026
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.

Can not add Source.Results to alerts

3 participants