-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[NEW-FEATURE] Poco types should be rather placed into onlinenamespace.Poco instead of Poco.onlinenamespace #342
Merged
PTKu
merged 17 commits into
dev
from
341-_NEW-FEATURE_Poco_types_should_be_rather_placed_into_onlinenamespace_Poco_instead_of_Poco_onlinenamespace
Nov 28, 2024
Merged
Conversation
This file contains 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
Introduce GetFullyQualifiedPocoName in CsHelpers class to return fully qualified POCO names. Update using directives to include AXSharp.Compiler.Cs.Helpers. Refactor Create methods in various builder classes to use GetFullyQualifiedPocoName. Update AddToSource method calls to use fully qualified POCO names. Modify CsPlainConstructorBuilder and CsPlainSourceBuilder to use GetFullyQualifiedPocoName. Wrap generated code in Pocos namespace in CsPlainSourceBuilder. Update AddCreatePocoMethod in CsOnlinerSourceBuilder to use GetFullyQualifiedPocoName. Adjust namespace and type declaration methods in CsPlainSourceBuilder to use GetFullyQualifiedPocoName.
Updated namespace references for Pocos classes across multiple files to use the global:: prefix. This change affects methods such as OnlineToPlainAsync, _OnlineToPlainNoacAsync, PlainToOnlineAsync, _PlainToOnlineNoacAsync, ShadowToPlainAsync, PlainToShadowAsync, DetectsAnyChangeAsync, and CreateEmptyPoco. The updates ensure the correct global namespace is used, avoiding potential conflicts and improving code maintainability.
Updated GitVersion.yml to set next-version to 0.20.0 and change mode for main branch. Increased segment sizes in WebApiConnector.cs. Modified AXSharp.Connector.S71500.WebAPITests.csproj to update package references and include new certificates. Replaced WebApiConnector instantiation with TestConnector.TestApiConnector in Exploratory.cs and WebApiConnectorTests.cs. Removed TestConnector class from WebApiPrimitiveTests.cs and added new using directives. Updated GH_PTKu_ix_56.cs, GH_PTKu_ix_59_68.cs, and GH_PTKu_ix_xx.cs to use TestConnector.SecurePlc. Updated apax.yml with new project name, version updates, environment variables, and scripts. Modified go.ps1 to set targetInput and update sld command. Updated Directory.Build.props to target net8.0. Added new TestConnector.cs for handling Web API connections and certificate validation.
…placed_into_onlinenamespace_Poco_instead_of_Poco_onlinenamespace
…d readability and consistency
Modified OnlinerBaseTypeTests.cs to use Is.InstanceOf for more accurate type checking in GetParentTest. Updated test_L10.ps1 and test_L2.ps1 scripts to run with --framework net9.0 and exit with $LASTEXITCODE for improved build and test processes.
Updated `dotnet run` command to use `--framework net9.0` in `dev.yml`, `master.yml`, `pr-dev.yml`, and `release.yml` to ensure the project runs with .NET 9.0. Removed the `Test Report` step using `dorny/test-reporter@v1` from all mentioned YAML files, which was generating a test report regardless of the previous steps' outcomes.
Updated the target framework version from .NET 8.0 to .NET 9.0 in multiple files: - In `BuildContext.cs`, changed `Framework` property of `DotNetRunSettings` from `"net8.0"` to `"net9.0"`. - In `AXSharp.templates.csproj`, updated `<TargetFramework>` element from `net8.0` to `net9.0`. - In `axsharpconsole.app.csproj`, updated `<TargetFramework>` element from `net8.0` to `net9.0`.
Updated `TestCollectionOrderer` attribute namespace in `DisplayNameOrderer.cs`. Removed various `Onliner` type properties and `Parent` and `Interpreter` properties across multiple classes and namespaces. Added methods `GetChildren`, `GetKids`, and `GetValueTags` to return collections of children, elements, and value tags. Updated `units.csproj` to target `net9.0` and updated package references.
Modified BuildContext to update ProcessRunner arguments and added new properties. Updated TestsTask to use >= for TestLevel and replaced AX_WEBAPI_TARGET with AXTARGET. Removed AXSharp-L3-tests.slnf. Added conditional compilation in ApaxTests. Updated TestConnector to use environment variables. Updated apax.yml with new targets, dependencies, and scripts. Enhanced Entry.cs with new methods and environment variables. Updated integrated.csproj to include new certificates. Removed conditional delays in PlainersSwappingTests and GH_ISSUE_183. Added new solution filters for integration and WebAPI tests. Added new certificate files.
PTKu
deleted the
341-_NEW-FEATURE_Poco_types_should_be_rather_placed_into_onlinenamespace_Poco_instead_of_Poco_onlinenamespace
branch
November 28, 2024 13:03
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.
This is major PR that addresses multiple issues including Poco namespace composition, removal of support of .net7,8 and porting to .net9; updates multiple packages. We also address some of the issue in the pipeline in particular with L3 testing. We bump here to v0.20.x
Description: Closes issue #341.
Changes:
Key Changes:
GetFullyQualifiedPocoName
method in various builder classes.global::
prefix.--framework net9.0
option in build scripts.Commits:
GetFullyQualifiedPocoName
method.global::
prefix.For more details, view the pull request.
closes #341