Conversation
There was a problem hiding this comment.
Pull request overview
This pull request adds Debian package support for Log4TC on Linux systems and attempts to upgrade the entire project from .NET 8.0 to .NET 10.0. The PR includes infrastructure for creating and hosting Debian packages via GitHub Pages using aptly, adds systemd service configuration, updates documentation with Linux installation instructions, and updates all NuGet package dependencies.
Key changes:
- Adds Debian package creation infrastructure with aptly configuration and Azure DevOps pipeline integration
- Attempts to upgrade all projects from .NET 8.0 to .NET 10.0 (critical issue: .NET 10.0 does not exist)
- Adds Linux installation documentation with apt repository setup instructions
Reviewed changes
Copilot reviewed 36 out of 39 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| source/aptly/* | New aptly configuration and database files for Debian repository management |
| source/Log4Tc/dockerfile | Updates framework references to net10.0 for test and publish stages |
| source/Log4Tc/**/*.csproj | Updates all project files from net8.0 to net10.0 target framework |
| source/Log4Tc/Mbc.Log4Tc.Service/Mbc.Log4Tc.Service.service | New systemd service unit file for Linux deployment |
| source/Log4Tc/Mbc.Log4Tc.Service/Mbc.Log4Tc.Service.csproj | Adds Packaging.Targets for Debian package creation and Linux path mappings |
| source/Log4Tc/Mbc.Log4Tc.SmokeTest/Mbc.Log4Tc.SmokeTest.csproj | Updates test SDK and assertion library versions |
| source/Log4Tc/Log4Tc.sln | Removes build configurations for setup project |
| source/Log4Tc/Directory.Build.props | Updates version to 25.12.14 and improves description |
| azure-pipelines-release.yml | Adds new stage for publishing Debian packages to GitHub Pages |
| docs/reference/installation.md | Adds comprehensive Linux installation section with apt repository configuration |
| docs/Changelog.md | Documents new Debian support and .NET 10.0 upgrade |
| .azure-pipelines/templates/log4tc/*.yaml | Updates pipeline templates to use net10.0 framework |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@copilot create a new devops pipeline to test the deb Installation of the latest version in an debian docker Container. This pipeline should be executed in each pr. Also start a Container with a setup influxdb that log4tc can connect to. All dies nformations can be found in the docs. Check if the connection is successfull. You will see it in the internal log. In future it should be possible to spin up a own twincat docker Container to run some end to end tests with the installed log4tc. |
Replaced FluentAssertions with AwesomeAssertions in all test projects and updated using statements. Upgraded FakeItEasy, Microsoft.NET.Test.Sdk, coverlet.collector, NLog, InfluxDB.Client, and System.Data.SqlClient to latest compatible versions. Updated Serilog dependencies and revised log file configuration to use monthly rolling files.
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
Log4tc Support linux debian with own feed and debian package also Update to .net10