Skip to content

Docstring for Web3 folder in Chainsafe.Gaming#674

Merged
kalambet merged 9 commits into
mainfrom
oleksandr/docstring-for-web3
Oct 25, 2023
Merged

Docstring for Web3 folder in Chainsafe.Gaming#674
kalambet merged 9 commits into
mainfrom
oleksandr/docstring-for-web3

Conversation

@oleksandrchainsafe
Copy link
Copy Markdown
Contributor

@oleksandrchainsafe oleksandrchainsafe commented Oct 19, 2023

plus some small refactoring

Some of the stuff was generated using ChatGPT while the other part was written by me. So if you find something is not descriptive enough, tell me.

Comment thread src/ChainSafe.Gaming/Web3/Core/Web3.cs Outdated
Comment thread src/ChainSafe.Gaming/Web3/Core/Build/Web3Builder.cs
Comment thread src/ChainSafe.Gaming/Web3/Core/Build/Web3Builder.cs
Comment thread src/ChainSafe.Gaming/Web3/Core/Build/Web3ServiceCollectionExtensions.cs Outdated
Comment thread src/ChainSafe.Gaming/Web3/Core/Build/Web3ServiceCollectionExtensions.cs Outdated
Comment thread src/ChainSafe.Gaming/Web3/Core/Web3.cs Outdated
Copy link
Copy Markdown
Contributor

@sneakzttv sneakzttv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yay comments

@kalambet kalambet dismissed robGG1997’s stale review October 25, 2023 13:26

It is a GitHub bug. Issue is already resolved.

@kalambet kalambet merged commit 77ee22d into main Oct 25, 2023
@kalambet kalambet deleted the oleksandr/docstring-for-web3 branch October 25, 2023 13:26
robGG1997 pushed a commit that referenced this pull request Nov 3, 2023
* Added XML comments for Web3 folder

* Added XML comments for Web3 folder (2)

* Added XML comments for Web3 folder (3)

* Fixed ChainRegistryProvider constructor

* Removed IInitializableWeb3

* Fixed Docstring for Web3 folder

* retrigger checks

---------

Co-authored-by: Peter Kalambet <peter@kalambet.dev>
rob1997 pushed a commit that referenced this pull request Jan 16, 2025
* Added XML comments for Web3 folder

* Added XML comments for Web3 folder (2)

* Added XML comments for Web3 folder (3)

* Fixed ChainRegistryProvider constructor

* Removed IInitializableWeb3

* Fixed Docstring for Web3 folder

* retrigger checks

---------

Co-authored-by: Peter Kalambet <peter@kalambet.dev>
sergeypanin1994 pushed a commit to sergeypanin1994/web3.unity that referenced this pull request Mar 16, 2025
* Added XML comments for Web3 folder

* Added XML comments for Web3 folder (2)

* Added XML comments for Web3 folder (3)

* Fixed ChainRegistryProvider constructor

* Removed IInitializableWeb3

* Fixed Docstring for Web3 folder

* retrigger checks

---------

Co-authored-by: Peter Kalambet <peter@kalambet.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Write Docsstring for Web3 folder in Chainsafe.Gaming. Analytics and probably Environment Folders are not important

4 participants