Skip to content

chore: upgrade EqDemoWinFormsNet6 to .NET 8.0#14

Open
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
upgrade/winforms-net6-to-net8
Open

chore: upgrade EqDemoWinFormsNet6 to .NET 8.0#14
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
upgrade/winforms-net6-to-net8

Conversation

@devin-ai-integration
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot commented Apr 7, 2026

Summary

Upgrades the EqDemoWinFormsNet6 WinForms project from .NET 6.0 to .NET 8.0:

  • TargetFramework: net6.0-windowsnet8.0-windows
  • Microsoft.EntityFrameworkCore.SqlServer: 6.0.18.0.11

Third-party packages (Korzh.EasyQuery.* 7.4.0, Korzh.DbUtils.* 1.4.x, EasyData.Exporters.* 1.5.9) were left unchanged as they target netstandard2.0 and are compatible with .NET 8.0.

Build verified on Linux with EnableWindowsTargeting=true — 0 errors, 1 pre-existing warning (unused field EasyQueryForm.ResultDataTable).

Review & Testing Checklist for Human

  • Verify EF Core 8.0 breaking changes don't affect this project — this jumps two major versions (6→8). Key areas: query translation changes, connection string handling, and any DateOnly/TimeOnly usage.
  • Smoke test on Windows — open the app, connect to a SQL Server database, run a query through the EasyQuery UI, and verify results load correctly. This was only build-verified (not runtime-tested).
  • Confirm .NET 8 SDK is available in any CI/CD or developer machines that build this project.

Notes

  • .NET 6 reached end-of-support in November 2024; .NET 8 is the current LTS (supported through November 2026).
  • The only warning emitted during build (CS0169: field 'EasyQueryForm.ResultDataTable' is never used) is pre-existing and unrelated to this upgrade.

Link to Devin session: https://app.devin.ai/sessions/d46a1862eadd497cb152484a15741ac8
Requested by: @tobydrinkall


Open with Devin

- Change TargetFramework from net6.0-windows to net8.0-windows
- Update Microsoft.EntityFrameworkCore.SqlServer from 6.0.1 to 8.0.11

Co-Authored-By: Toby Drinkall <toby.drinkall@cognition.ai>
@devin-ai-integration
Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link
Copy Markdown
Author

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

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