feat!: restructure SDK with multi-targeting, test suites and modern HTTP/logging - #8
Open
Renancp01 wants to merge 1 commit into
Open
feat!: restructure SDK with multi-targeting, test suites and modern HTTP/logging#8Renancp01 wants to merge 1 commit into
Renancp01 wants to merge 1 commit into
Conversation
…TTP/logging
- Multi-target net8.0;net10.0 (existing net8 consumers keep working)
- Rename projects/solution to .NET naming convention (Inter.Sdk*)
- Remove orphan netstandard2.0 project (src/, inter-sdk-csharp.csproj)
- Stop tracking build artifacts and IDE folders (bin/, obj/, .idea/)
with proper .gitignore coverage at every level
- Add Inter.Sdk.UnitTests (29 tests) covering enums, config, URL
building, error parsing, model serialization and logging
- Add WireMock-based Inter.Sdk.IntegrationTests (62 tests) covering
every public client method end-to-end without credentials, plus an
optional sandbox suite gated by environment credentials
- Migrate HTTP layer from obsolete HttpWebRequest (SYSLIB0014) to
pooled HttpClient, preserving public signatures, 429 retry and
error parsing behavior
- Add pluggable logging via Microsoft.Extensions.Logging (ILogger
injection); request/response bodies log at Debug level; token
responses are never logged; default file logger keeps the
historical logs/inter-sdk-{Day}.log behavior
- Add optional Config.BaseUrl override (tests, proxies)
- Add English README and NuGet publishing workflow (tag v* -> nuget.org)
BREAKING CHANGE: the root namespace changed from inter_sdk_library to
Inter.Sdk (standard .NET PascalCase naming) and the assembly/package is
now Inter.Sdk (2.0.0) instead of inter-sdk-library. Consumers must
replace 'using inter_sdk_library;' with 'using Inter.Sdk;' and update
DLL/package references to the new name.
Renancp01
marked this pull request as ready for review
August 18, 2026 01:10
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
net8.0;net10.0— existing .NET 8 consumers keep working, new ones get .NET 10Inter.Sdk,Inter.Sdk.UnitTests,Inter.Sdk.IntegrationTests,Inter.Sdk.FunctionalTests)src/,inter-sdk-csharp.csproj) and stop tracking build artifacts (bin/,obj/,.idea/)INTER_*environment variablesHttpWebRequest(SYSLIB0014) to pooledHttpClient, preserving public signatures, 429 retry and error parsing behaviorMicrosoft.Extensions.Logging: inject anyILogger(Serilog/Loki, OpenTelemetry, console...); request/response bodies log at Debug level; token responses are never logged; default file logger preserves the historicallogs/inter-sdk-{Day}.logbehaviorConfig.BaseUrloverride (tests, proxies)v*→ nuget.org, requiresNUGET_API_KEYsecret)Root namespace changed from
inter_sdk_librarytoInter.Sdk(standard .NET PascalCase naming) and the assembly/package is nowInter.Sdk2.0.0 instead ofinter-sdk-library.Migration for consumers:
DLL/package references must point to the new
Inter.Sdkname.Open questions for maintainers
Inter.Sdkavailability / reserved prefix on nuget.org and which org account holds the publish keyTest evidence
dotnet test Inter.Sdk.UnitTests→ 29 passed (net8.0 and net10.0)dotnet test Inter.Sdk.IntegrationTests→ 62 passed + 1 skipped sandbox (net8.0 and net10.0)dotnet pack Inter.Sdk -c Release→Inter.Sdk.2.0.0.nupkgbuilds cleanly