Skip to content

Commit

Permalink
Merging new changes from master into azconfig (#3076)
Browse files Browse the repository at this point in the history
* Made some classes/fields private

* Block blob tests

* Changing method of comparing flowable data to buffer list

* Only fromFile and fromBuffer tests remaining

* Finished up transfer manager tests

* Resolved some CR comments

* CR feedback

* Updated some project infrastructure. Fixed some failing tests. Three tests still failing, but they are not critical

* Added some missing licenses to tests

* Started writing retry policy

* Implemented first test scenario. Not tested.

* Fixed up some container tests that weren't actually validating properly. Got the first retry test working.

* Added test for max retries and made fixes that now correctly capture errors

* Got a bit more of the delay test working

* All retry tests passing

* Some cleanup

* Minor fixes

* CR feedback

* Moved some files around. Made getters on RetryOptions package private

* Removed an extra file left over from a merge

* Update README.md

Adding direct link to the API Reference docs for the Preview release and quickstart

* Tried making some swagger changes

* Another iteration of swagger generation

* Everything broken

* Generated swagger with proper (de)serialization

* Added support for setBlobTier. No tests or samples

* Tests for blob tier

* CR feedback

* Make AzureStorageCheckpointLeaseManager sync internally to get around potential deadlock when checkpointing (#349)

* Doc for append blob create

* Updated file reference

* Added apiNote

* Fixed sample ref path again

* Addressing the repo root as ..

* Started on ref samples. Maybe a third of methods done.

* Moved the sample tags to the bottom

* added a few more samples

* More samples

* More samples

* More samples

* PageBlob Samples

* Added lease samples

* api ref samples complete

* soft delete support. no tests or samples yet

* Changelog

* Working on tests

* Soft delete tests

* Changelog and BreakingChanges

* CR feedback

* Added api ref samples for tiers and soft delete

* Fixed swagger README

* Update EPH dependences. Update EPH version for release. (#353)

* Fixed an api note to generate docs

* Appveyor is running EPH tests that should not run (#356)

* Appveyor is running EPH tests that should not run.
The tests only run if certain environment variables are set.
I don't know how or why they are set. Force Appveyor to unset them.

* Maven 3.5.2 doesn't exist on apache servers anymore, update to 3.5.4.

* PutBlockFromURL

* Improved ContainerURL tests

* Updated api ref samples

* Cleaned up some types in tests that were causing failures

* Appveyor detection fix (#358)

EPH test cases which need actual Azure entities look in environment variables for the connection strings and skip if not found. Those environment variables shouldn't be set in Appveyor but the test cases were trying to run anyway, and failing. Syntax for appveyor.yml allows for setting environment variables but it isn't obvious how to delete them, so change code to also check APPVEYOR environment variable.

* Fixing API ref links

* 10.0.2-Preview release prep

* Updated dependency and javadoc configuration

* Generated with new autorest version to comply with new runtime

* Updated dependency issue. Releasing 10.0.3-Preview to fix

* EPH 2.0.1 release -- fix deadlocks (#357)

Due to a threading change in client 1.0.2, EPH 2.0.0 has the potential to deadlock due to thread starvation under a particular set of conditions:

* The Event Hub has more partitions than the threadpool has threads
* AND the application is using the default Azure Storage-based checkpoint manager
* AND the IEventProcessor.onEvents implementation checkpoints
* AND a large number of partitions on the same host attempt to checkpoint at almost the same time.

The problem: The threading change in client 1.0.2 means that IEventProcessor.onEvents is executing in the same threadpool where the checkpoint manager runs. To ensure ordering of checkpoints, it is important that the onEvents implementation waits for the checkpoint call to return, which means a thread in the threadpool is blocked. If there are too many checkpoint calls in rapid succession, it is possible for all threads in the threadpool to be blocked, leaving no threads available to perform the checkpointing.

The fix: This condition occurred because the Azure Storage-based checkpoint manager was attempting to be asynchronous internally, even though the API exposed by Azure Storage is synchronous. Thus, writing one checkpoint required two threads: one running onEvents and blocking, and another to run the task that calls Azure Storage. Since the Azure Storage API is synchronous, this is wasteful at best. Refactored the default checkpoint manager to be mostly synchronous and return pre-completed CompletableFutures. This means that onEvents and the Storage calls run on the same thread. That thread still ends up blocked waiting for Storage, but even if every thread in our threadpool ends up blocked that way, the Storage client has its own separate threadpool internally and can always make progress. The Storage calls will always return (successfully or not) and unblock our threads.

* Distinguish between nonexistent entity and other errors which return amqp:not-found (#354)

* Update how to add properties (#361)

* Websockets support (#362)

1. Supports max_frame_size of 4k (current limitation of <a href="https://github.com/Azure/qpid-proton-j-extensions">qpid-proton-j-extensions</a> library)

2. doesn't support PROXY on websockets - this will follow.

Amqp over WebSockets is particularly used when enterprise policies (ex: firewall outbound port rules) doesn't allow traffic on the default Amqp secure port (`5671`).
To send or receive over websockets - which uses port `443`, set the `TransportType` on `ConnectionStringBuilder`, like this:

```
connectionStringBuilder.setTransportType(TransportType.AmqpWebSockets)
```

related: #264

* Update Overview.md (#363)

* Fix params

* Added worm support

* getAccountInfo apis added

* Improved append blob tests

* Static websites

* Updated blob tests

* Most page blob tests

* Retry fixes

* Fix Overview files & update proton library version (#365)

* update API in the overview docs to work with latest version.
* update proton version.

* Service SAS Signature Values correctly handles nulllable fields

* Page blob tests updates

* Fixing links for API reference

* Got some boilerplate going

* Got some preliminary tests written and passing

* Added some tests

* downloadToBuffer complete

* downloadBlobToFile written

* Nothing really

* Basic test passing

* downloadToFileTests complete

* Moved sample tag

* added a cast to prevent a potential overflow

* Added etag locking

* Made the options properties final as possible

* CR feedback

* Nothing special

* Fixed issue in etag locking that messed up access conditions

* Removed memory mapping from TransferManager

* CR feedback

* ServiceSAS tests

* SAS tests complete

* Updated links for api refs

* Logging tests

* Updated changelog

* Added a test to ensure uploadFromFile is replayable

* Last minute cleanup before release

* Fixed some transiet failures in incremental copy tests

* RequestResponseChannel should not throw OperationCancelled. (#372)

* Implement retry mechanism for Receiver and Sender creation (#373)

* msgreceiver creation - retry on transient errors
* messagesender creation retry
* always throw TimeoutException on transient failures
* throw schedulererrors to usercode
* verify retry invocation in unittests

* update version for releasing version 1.1.0 (#374)

* Release commit for package com.microsoft.azure.eventhubs version 1.1.0  (#375)

* websocket support (#362)
* fix an issue with ExceptionContract - when request-response channel closes with transient error (#372)
* include PartitionReceiver and PartitionSender creation to participate in RetryPolicy (#373)

* Interfaces for helper types updated

* Added reliable download functionality to DownloadResponse

* Most tests converted to downloadResponse

* Removed RetryReader

* Cleanup

* Reverted to retrying on IOException

* Small doc update

* Added context class

* Switched to autorest generated headers and access conditions

* Fixed tests

* Extracted TransferManager inner classes to top level

* Added hints to user that Flowables for upload must be replayable

* Incorporated new autorest exception type

* Updated some tests

* Changelog breakingChanges

* Generated with context feature

* Added context parameter. need to add tests

* Started context tests

* Infra for context tests

* Halfway through context tests

* Added tests for context

* Changelog breakingChanges

* Deleted some old files

* Fixed a broken test

* Release cleanup

* Added min overloads

* WebSockets Proxy support (#378)

* followup: websockets proxy support (#381)

* Cleaned up and reformatted some docs immediately postGA. Trying establish consistent style. Other misc cleanup

* Added some test infra and logging to help debug errors in CI

* Added package info for blob package for javadoc description

* Fixed up api ref samples link and CI

* Added deep sync copy support

* Added support for progress reporting

* Fix failing unit tests (#381)

Fix some unit tests to no longer depend on specific test environment.

* Fixed some overflow bugs in TransferManager and reliable download setup in BlobURL

* CR feedback and updated Contributing doc

* Added issue template

* Added min overloads

* Cleaned up and reformatted some docs immediately postGA. Trying establish consistent style. Other misc cleanup

* Added some test infra and logging to help debug errors in CI

* Added package info for blob package for javadoc description

* Fixed up api ref samples link and CI

* Added deep sync copy support

* Fixed some overflow bugs in TransferManager and reliable download setup in BlobURL

* Added support for progress reporting

* CR feedback and updated Contributing doc

* Release prep

* * Fixed a de-dup - of Delivery event issue on proton receivelink (#391)

* Updated LoggingFactory to also log request HTTP method, URL, and headers (#394)

* Updated LoggingFactory to also log request HTTP method, URL, and headers

* Updated README to include table

* Update README.md

* Update README.md

* Update version numbers for EPH 2.0.2 release (#394)

* Fixed a flaky block blob test

* Added policy to set request on response object

* Added custom deserializer to fix bug that would sometimes return incomplete listing results

* Added support for slf4j. Added default logging

* 10.3.0 release prep

* Update ChangeLog.txt

* Update ChangeLog.txt

* EPH PartitionPump fixes (#401)

* Handle tmpdir with no trailing separator.

* Fix IO pipe stuck issue due to aggressive timer scheduling (#402)

There is an issue in the logic where operation timeout timer is scheduled as part of a receive call and due to the bug operation timer can be scheduled multiple times although it is supposed to be set only once when there is no pending receive call. Over time as the operation timer keeps getting scheduled during receive API call and scheduled again by the reactor thread while the event is fired and being handled, the number of pending IO operations on the pipe gets incremented significantly and it can cause IO pipe stuck and blocks a write operation on the pipe, which in turn, blocks a receive API. There are two changes to address the issue: a) ensure that operation timer is scheduled at most two times to avoid excessive IO operations on the pipe and b) read all bytes from the channel when signaled so that there are no remaining bytes in the channel.

* Prep for releasing client 1.3.0 and EPH 2.1.0 (#403)

* Release client 2.0.0 and EPH 2.2.0 (#416)

* Update Apache Proton-J dependency (0.29.0 --> 0.31.0) (#407)

* PartitionReceiver - add a method that provides an EventPosition which corresponds to an EventData returned last by the receiver (#408)

* Support IsPartitionEmpty property for PartitionRuntimeInformation (#399)

* Move setPrefetchCount API to the ReceiverOptions class from the PartitionReceiver and update the settings of Default & Max Prefetch count (#410)

This pull request includes two major changes related to Prefetch API.

1) Move setPrefetchCount API to the ReceiverOptions class so that prefetch value specified by a user can be used instead of using default value when communicating to the service during link open and initializing a receiver. This change also addresses the receiver stuck issue caused by setPrefetchAPI in a race condition.

2) Change the default value and set the upper bound of the prefetch count. Note that prefetch count should be greater than or equal to maxEventCount which can be set when either a) calling receive() API or b) implementing the getMaxEventCount API of the SessionReceiverHandler interface.

* Fixes several issues in the reactor related components (#411)

This pull request contains the following changes.

1) Finish pending tasks when recreating the reactor and make sure pending calls scheduled on the old reactor get complete.
2) Fix the session open timeout issue which can result in NPE in proton-J engine.
3) Make session open timeout configurable and use the value of OperationTimeout.
4) Update the message of exceptions and include an entity name in the exception message.
5) API change - use ScheduledExecutorService.
6) Improve tracing.

* Implement comparable on EventData (#395)

* Update receive/send link creation logic and improve tracing (#414)

* Prep for releasing client 2.0.0 and EPH 2.2.0 (#415)

* Removed some lingering dev notes

* Changed test behavior to allow for absence of some creds

* Release prep

* Upload from non replayable flowable

* 10.5.0 release prep

* Standardize indention and publishing (#2974)

* SpotBug P3 Fixes (#2973)

* Adding suppression for having named static inner classes from anonymous.

* Add suppression for redundant null check.

* Move SIC_INNER_SHOULD_BE_STATIC suppression

* Make anonymous class in CachingKeyResolver a named inner static.

* Include missing impl classes for auto-rest when it generates a variable with known null value

* Suppressing unconfirmed casts because we know the cast is correct but the APIs weren't designed for it

* Bumping up resource-manager parent and eventgrid.v2019_01_01 to version 1.0.0 (#2975)

* Bumping up resource-manager parent pom to 1.0.0

* Bumping up eventgrid.v2019_01_01 to version 1.0.0

* code restructure

* [AutoPR privatedns/resource-manager] Add private dns module name for java codegen (#3009)

* Generated from e92e525831933bedf2242fe607984757387be1b2

Add private dns module name for java codegen

* Update pom for private dns

* change private dns version back to beta

* SpotBug P3 fixes (#2976)

* Suppress using Nullable parameter as non-null for KeyVaultKey because it was a scenario in original SDK not thought of.

* Suppress return empty array rather than null

* Add null check for service to ensure it has been initialized

* Checking for null service value before invoking method

* Catching specific exceptions rather than Exception e

* Use 8L so that the integer multiplication is not implicitly cast to long.

* Add exclude for catch Exception in CachingKeyResolver

* Catching specific exceptions rather than Exception e

* Update YAML formatting (#3033)

- Update whitespace to match Prettier formatting
- Add .prettierrc.yml
- Part of Azure/azure-sdk#225

* adding impression pixel (#3024)

license/cla override. Check isn't coming back for some reason.

* Add .editorconfig and .gitattributes (#3036)

* Adding .editorconfig

* Add .gitattributes

* Reformat pom.client.xml

* Reformat keyvault/data-plane/pom.xml

* Reformat batch/data-plane/pom.xml

* Enable Checkstyle fail on error / violation for Key Vault (#2969)

* Now that KeyVault data-plane has no more checkstyle errors or violations, we are now enforcing build failure for this project and all child modules.

* Install build tools locally so that checkstyle can run.

* Fix whitespace

* It is not necessary to run checkstyle:check as the build will fail satisfactorily under the analyze job.

* Record playback integration (#2870)

Record/Playback Integration in Azure Batch Java SDK tests. 
Added Test recordings to be used in playback mode.

* Normalising lines (#3050)

* Fix for issue #2874. Add a cache for the tokens in case we are getting (#2875)

tokens from IMDS.

* Run dependency checker during Analyze (#3075)

* Run dependency checker during Analyze

* Add docs links
  • Loading branch information
conniey authored Mar 10, 2019
1 parent 1525dc7 commit 0681574
Show file tree
Hide file tree
Showing 458 changed files with 74,591 additions and 3 deletions.
18 changes: 18 additions & 0 deletions .azure-pipelines/client.yml
Original file line number Diff line number Diff line change
Expand Up @@ -95,10 +95,28 @@ jobs:
publishJUnitResults: false
goals: 'site:site site:stage'

- script: |
git clone https://github.com/JonathanGiles/DependencyChecker.git
mkdir input && cp dependencies.json input/
displayName: 'Download dependency checker'
- task: Maven@3
displayName: 'Analyze dependencies'
inputs:
mavenPomFile: 'DependencyChecker/pom.xml'
options: '-Dexec.args="-showall"'
mavenOptions: '$(LoggingOptions)'
javaHomeOption: 'JDKVersion'
jdkVersionOption: '1.11'
jdkArchitectureOption: 'x64'
publishJUnitResults: false
goals: 'clean package exec:java'

- powershell: |
copy -r target/staging $(Build.ArtifactStagingDirectory)
copy eng/code-quality-reports/src/main/resources/index.html $(Build.ArtifactStagingDirectory)
copy eng/spotbugs-aggregate-report/target/spotbugs/spotbugsXml.html (Join-Path $(Build.ArtifactStagingDirectory) "staging")
copy output/dependencies.html (Join-Path $(Build.ArtifactStagingDirectory) "staging")
displayName: 'Copy reports to artifact staging'
- task: PublishBuildArtifacts@1
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
| Component | Build Status |
| --------- | ------------ |
| Management Libraries | [![Build Status](https://travis-ci.org/Azure/azure-sdk-for-java.svg?branch=master)](https://travis-ci.org/Azure/azure-sdk-for-java) |
| Client Libraries | [![Build Status](https://dev.azure.com/azure-sdk/public/_apis/build/status/17?branchName=master)](https://dev.azure.com/azure-sdk/public/_build/latest?definitionId=17) |
| Client Libraries | [![Build Status](https://dev.azure.com/azure-sdk/public/_apis/build/status/17?branchName=master)](https://dev.azure.com/azure-sdk/public/_build/latest?definitionId=17)<br>[![Build Documentation](https://img.shields.io/badge/documentation-published-blue.svg)](https://azuresdkartifacts.blob.core.windows.net/azure-sdk-for-java/index.html)|

:+1: [Try Azure for FREE](http://go.microsoft.com/fwlink/?LinkId=330212)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@
import java.text.ParseException;
import java.text.SimpleDateFormat;
import java.util.*;
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.locks.Lock;
import java.util.concurrent.locks.ReentrantLock;

/**
* Managed Service Identity token based credentials for use with a REST Service Client.
Expand All @@ -24,6 +27,9 @@ public final class MSICredentials{
//
private final List<Integer> retrySlots = new ArrayList<>();
//
private final Lock lock = new ReentrantLock();

private final ConcurrentHashMap<String, MSIToken> cache = new ConcurrentHashMap<>();
private final MSIConfigurationForVirtualMachine configForVM;
private final MSIConfigurationForAppService configForAppService;
private final HostType hostType;
Expand Down Expand Up @@ -157,7 +163,7 @@ public void updateObjectId(String objectId) {
public MSIToken getToken(String tokenAudience) throws IOException, AzureMSICredentialException{
switch (hostType) {
case VIRTUAL_MACHINE:
return this.retrieveTokenFromIDMSWithRetry(tokenAudience == null ? this.configForVM.resource() : tokenAudience);
return this.getTokenForVirtualMachineFromIMDSEndpoint(tokenAudience == null ? this.configForVM.resource() : tokenAudience);
case APP_SERVICE:
return this.getTokenForAppService(tokenAudience);
default:
Expand Down Expand Up @@ -217,6 +223,51 @@ private MSIToken getTokenForAppService(String tokenAudience) throws IOException,
}
}

private MSIToken getTokenForVirtualMachineFromIMDSEndpoint(String tokenAudience) throws AzureMSICredentialException {
String tokenIdentifier = tokenAudience;

String extraIdentifier = null;
if (this.configForVM.objectId() != null)
{
extraIdentifier = configForVM.objectId();
} else if (this.configForVM.clientId() != null) {
extraIdentifier = configForVM.clientId();
} else if (this.configForVM.identityId() != null) {
extraIdentifier = configForVM.identityId();
}

if (extraIdentifier != null) {
tokenIdentifier = tokenIdentifier + "#" + extraIdentifier;
}

MSIToken token = cache.get(tokenIdentifier);
if (token != null && !token.isExpired()) {
return token;
}

lock.lock();

try {
token = cache.get(tokenIdentifier);
if (token != null && !token.isExpired()) {
return token;
}

try {
token = retrieveTokenFromIDMSWithRetry(tokenAudience);
if (token != null) {
cache.put(tokenIdentifier, token);
}
} catch (IOException exception) {
throw new AzureMSICredentialException(exception);
}

return token;
} finally {
lock.unlock();
}
}

private MSIToken retrieveTokenFromIDMSWithRetry(String tokenAudience) throws AzureMSICredentialException, IOException {
StringBuilder payload = new StringBuilder();
final int imdsUpgradeTimeInMs = 70 * 1000;
Expand Down
6 changes: 6 additions & 0 deletions dependencies.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
[
{
"projectName": "azure-sdk-for-java",
"pomUrls": ["file:./pom.client.xml"]
}
]
4 changes: 4 additions & 0 deletions eng/code-quality-reports/src/main/resources/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -499,6 +499,10 @@ <h2>Azure Java SDK</h2>
<ion-icon name="book"></ion-icon>
<a href="staging/index.html" target="_blank" class="links">Maven Site</a>
</li>
<li class="list-group-item sub-component">
<ion-icon name="clipboard"></ion-icon>
<a href="staging/dependencies.html" target="_blank" class="links">Dependencies</a>
</li>
<li class="list-group-item sub-component">
<ion-icon name="bug"></ion-icon>
<a href="staging/spotbugsXml.html" target="_blank" class="links">SpotBugs (Aggregate)</a>
Expand Down
26 changes: 26 additions & 0 deletions eventhubs/data-plane/.gitattributes
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
# Default behavior: if Git thinks a file is text (as opposed to binary), it
# will normalize line endings to LF in the repository, but convert to your
# platform's native line endings on checkout (e.g., CRLF for Windows).
* text=auto

# Explicitly declare text files you want to always be normalized and converted
# to native line endings on checkout. E.g.,
#*.c text

# Declare files that will always have CRLF line endings on checkout. E.g.,
#*.sln text eol=crlf

# Declare files that will always have LF line endings on checkout. E.g.,
*.sh text eol=lf

# Denote all files that should not have line endings normalized, should not be
# merged, and should not show in a textual diff.
*.docm binary
*.docx binary
*.ico binary
*.lib binary
*.png binary
*.pptx binary
*.snk binary
*.vsdx binary
*.xps binary
50 changes: 50 additions & 0 deletions eventhubs/data-plane/.github/CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,50 @@
# Contribute or Provide Feedback for Azure Event Hubs Java library

## Table of Contents

- [Code of Conduct](#code-of-conduct)
- [Filing Issues](#filing-issues)
- [Pull Requests](#pull-requests)
- [General guidelines](#general-guidelines)
- [Testing guidelines](#testing-guidelines)

## Code of Conduct

This project has adopted the [Microsoft Open Source Code of Conduct](https://opensource.microsoft.com/codeofconduct/). For more information see the [Code of Conduct FAQ](https://opensource.microsoft.com/codeofconduct/faq/) or contact [opencode@microsoft.com](mailto:opencode@microsoft.com) with any additional questions or comments.

## Filing Issues

You can find all of the issues that have been filed in the [Issues](https://github.com/Azure/azure-event-hubs-java/issues) section of the repository.

If you encounter any bugs, please file an issue [here](https://github.com/Azure/azure-event-hubs-java/issues/new) and make sure to fill out the provided template with the requested information.

To suggest a new feature or changes that could be made, file an issue the same way you would for a bug, but remove the provided template and replace it with information about your suggestion.

### Pull Requests

If you are thinking about making a large change to this library, **break up the change into small, logical, testable chunks, and organize your pull requests accordingly**.

You can find all of the pull requests that have been opened in the [Pull Request](https://github.com/Azure/azure-event-hubs-java/pulls) section of the repository.

To open your own pull request, click [here](https://github.com/Azure/azure-event-hubs-java/compare). When creating a pull request, keep the following in mind:
- Make sure you are pointing to the fork and branch that your changes were made in
- The pull request template that is provided **should be filled out**; this is not something that should just be deleted or ignored when the pull request is created
- Deleting or ignoring this template will elongate the time it takes for your pull request to be reviewed

#### General guidelines

The following guidelines must be followed in **EVERY** pull request that is opened.

- Title of the pull request is clear and informative
- There are a small number of commits that each have an informative message
- A description of the changes the pull request makes is included, and a reference to the bug/issue the pull request fixes is included, if applicable
- All files have the Microsoft copyright header

#### Testing guidelines

The following guidelines must be followed in **EVERY** pull request that is opened.

- Pull request includes test coverage for the included changes
- Tests must use xunit
- Test code should not contain hard coded values for resource names or similar values
- Test should not use App.config files for settings
11 changes: 11 additions & 0 deletions eventhubs/data-plane/.github/ISSUE_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
## Actual Behavior
1.
2.

## Expected Behavior
1.
2.

## Versions
- OS platform and version:
- Maven package version or commit ID:
16 changes: 16 additions & 0 deletions eventhubs/data-plane/.github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
## Description
<!--
Please add an informative description that covers the changes made by the pull request.
If applicable, reference the bug/issue that this pull request fixes here.
-->

This checklist is used to make sure that common guidelines for a pull request are followed.

- [ ] **I have read the [contribution guidelines](./CONTRIBUTING.md).**
- [ ] Title of the pull request is clear and informative.
- [ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR.
- [ ] The pull request does not introduce breaking changes (unless a major version change occurs in the assembly and module).
- [ ] If applicable, the public code is properly documented.
- [ ] Pull request includes test coverage for the included changes.
- [ ] The code builds without any errors.
Loading

0 comments on commit 0681574

Please sign in to comment.