Skip to content

Make the client using streams the default. Using strings is not sustainable #2303

Make the client using streams the default. Using strings is not sustainable

Make the client using streams the default. Using strings is not sustainable #2303

Triggered via pull request October 7, 2024 08:50
Status Failure
Total duration 2m 13s
Artifacts

ci.yml

on: pull_request
Matrix: Integration tests
Matrix: Unit tests
License update
0s
License update
Create artifacts
0s
Create artifacts
Fit to window
Zoom out
Zoom in

Annotations

1 error and 44 warnings
Integration tests (Stream)
Process completed with exit code 1.
Integration tests (Stream)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Integration tests (Stream): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L44
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Integration tests (Stream): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L68
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Integration tests (Stream): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L131
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Integration tests (Stream): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L151
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Integration tests (Stream): commercetools.Sdk/compat/commercetools.Sdk.V2Compat/CtpV2CompatClient.cs#L44
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
Integration tests (Stream): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/OrderPatches/ReturnShipmentState.cs#L40
The member 'ReturnShipmentStateWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Integration tests (Stream): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Orders/CartOrigin.cs#L34
The member 'CartOriginWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Integration tests (Stream): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/ImportResourceType.cs#L70
The member 'ImportResourceTypeWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Integration tests (Stream): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/MoneyType.cs#L34
The member 'MoneyTypeWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Integration tests (Stream): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/ProcessingState.cs#L49
The member 'ProcessingStateWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (macos-latest)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Unit tests (macos-latest): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L44
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Unit tests (macos-latest): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L68
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Unit tests (macos-latest): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L131
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Unit tests (macos-latest): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L151
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Unit tests (macos-latest): commercetools.Sdk/compat/commercetools.Sdk.V2Compat/CtpV2CompatClient.cs#L44
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
Unit tests (macos-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Importoperations/ImportOperationState.cs#L34
The member 'ImportOperationStateWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (macos-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/ImportResourceType.cs#L70
The member 'ImportResourceTypeWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (macos-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/MoneyType.cs#L34
The member 'MoneyTypeWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (macos-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/ProcessingState.cs#L49
The member 'ProcessingStateWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (macos-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/ProductPriceModeEnum.cs#L34
The member 'ProductPriceModeEnumWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (ubuntu-latest)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Unit tests (ubuntu-latest): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L44
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Unit tests (ubuntu-latest): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L68
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Unit tests (ubuntu-latest): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L131
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Unit tests (ubuntu-latest): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L151
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Unit tests (ubuntu-latest): commercetools.Sdk/compat/commercetools.Sdk.V2Compat/CtpV2CompatClient.cs#L44
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
Unit tests (ubuntu-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Importoperations/ImportOperationState.cs#L34
The member 'ImportOperationStateWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (ubuntu-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/OrderPatches/ReturnShipmentState.cs#L40
The member 'ReturnShipmentStateWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (ubuntu-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/ProcessingState.cs#L49
The member 'ProcessingStateWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (ubuntu-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/MoneyType.cs#L34
The member 'MoneyTypeWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (ubuntu-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Producttypes/AttributeConstraintEnum.cs#L40
The member 'AttributeConstraintEnumWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (windows-latest)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Unit tests (windows-latest): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L44
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Unit tests (windows-latest): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L68
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Unit tests (windows-latest): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L131
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Unit tests (windows-latest): commercetools.Sdk/commercetools.Base.Client/Tokens/TokenProvider.cs#L151
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
Unit tests (windows-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/ImportResourceType.cs#L70
The member 'ImportResourceTypeWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (windows-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/MoneyType.cs#L34
The member 'MoneyTypeWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (windows-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/ProcessingState.cs#L49
The member 'ProcessingStateWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (windows-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/ProductPriceModeEnum.cs#L34
The member 'ProductPriceModeEnumWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (windows-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Common/ReferenceType.cs#L88
The member 'ReferenceTypeWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.
Unit tests (windows-latest): commercetools.Sdk/commercetools.Sdk.ImportApi/Generated/commercetoolsSdkImportApi/Models/Customers/AuthenticationMode.cs#L34
The member 'AuthenticationModeWrapper.GetEnumerator()' does not hide an accessible member. The new keyword is not required.