Skip to content
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

refactor(arch3-core)!: rewrite all queries in the ArchwayClient #58

Merged
merged 5 commits into from
Apr 28, 2023

Conversation

aelesbao
Copy link
Member

@aelesbao aelesbao commented Apr 28, 2023

The queries did not conform to the CosmWasmClient API. It used static methods and exposed unnecessary internal interfaces from the @archwayhq/arch3-proto package.

This refactor simplifies the API usage and isolates the querying interface to a separate module.

Some linting rules also changed:

  • lint jsdoc
  • import sorting rules
  • enforce single quotes
  • parallelize the linting tasks to reduce the execution time

- lint jsdoc
- import sorting rules
- enforce single quotes
- parallelize the linting tasks to reduce the execution time
The queries did not conform to the `CosmWasmClient` API. It used static
method and it exposed unnecessary internal interfaces from the
`@archwayhq/arch3-proto` package.

This refactor simplifies the API usage and isolates the querying
interface to a separate module.
* @returns An offline `SigningArchwayClient`.
*/
// eslint-disable-next-line @typescript-eslint/require-await
public static override async offline(

Check failure

Code scanning / ESLint

Disallow async functions which have no `await` expression

Static async method 'offline' has no 'await' expression.
@aelesbao aelesbao merged commit 4a3be6b into main Apr 28, 2023
@aelesbao aelesbao deleted the refactor/archwayclient-queries branch April 28, 2023 15:28
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.

1 participant