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

Feat/272 split sdk and postman #381

Merged
merged 32 commits into from
Dec 9, 2024
Merged

Feat/272 split sdk and postman #381

merged 32 commits into from
Dec 9, 2024

Conversation

VGau
Copy link
Collaborator

@VGau VGau commented Dec 3, 2024

This PR implements issue(s) #272

Checklist

  • I wrote new tests for my new core changes.
  • I have successfully ran tests, style checker and build against my new changes locally.
  • I have informed the team of any breaking changes if there are any.

@VGau VGau had a problem deploying to docker-build-and-e2e December 3, 2024 08:12 — with GitHub Actions Error
@VGau VGau requested a review from jonesho December 3, 2024 08:38
@VGau VGau had a problem deploying to docker-build-and-e2e December 3, 2024 09:05 — with GitHub Actions Error
@VGau VGau had a problem deploying to docker-build-and-e2e December 3, 2024 09:11 — with GitHub Actions Error
@codecov-commenter
Copy link

codecov-commenter commented Dec 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.17%. Comparing base (4e93fa3) to head (02af574).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #381   +/-   ##
=========================================
  Coverage     70.17%   70.17%           
  Complexity     1070     1070           
=========================================
  Files           306      306           
  Lines         12337    12337           
  Branches       1179     1179           
=========================================
  Hits           8658     8658           
  Misses         3200     3200           
  Partials        479      479           
Flag Coverage Δ *Carryforward flag
hardhat 98.70% <ø> (ø) Carriedforward from d37340b
kotlin 67.86% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.

@VGau VGau had a problem deploying to docker-build-and-e2e December 3, 2024 11:36 — with GitHub Actions Error
@VGau VGau force-pushed the feat/272-split-sdk-and-postman branch from f114b77 to 40fcef1 Compare December 3, 2024 11:44
@VGau VGau had a problem deploying to docker-build-and-e2e December 3, 2024 11:45 — with GitHub Actions Error
@VGau VGau had a problem deploying to docker-build-and-e2e December 3, 2024 11:48 — with GitHub Actions Error
@VGau VGau temporarily deployed to docker-build-and-e2e December 3, 2024 15:27 — with GitHub Actions Inactive
@VGau VGau temporarily deployed to docker-build-and-e2e December 3, 2024 16:58 — with GitHub Actions Inactive
@VGau VGau had a problem deploying to docker-build-and-e2e December 5, 2024 13:50 — with GitHub Actions Error
@VGau VGau had a problem deploying to docker-build-and-e2e December 5, 2024 13:53 — with GitHub Actions Error
@VGau VGau had a problem deploying to docker-build-and-e2e December 5, 2024 14:29 — with GitHub Actions Error
@VGau VGau temporarily deployed to docker-build-and-e2e December 5, 2024 14:30 — with GitHub Actions Inactive
@VGau VGau had a problem deploying to docker-build-and-e2e December 6, 2024 11:57 — with GitHub Actions Error
@VGau VGau temporarily deployed to docker-build-and-e2e December 6, 2024 12:02 — with GitHub Actions Inactive
@VGau VGau had a problem deploying to docker-build-and-e2e December 6, 2024 12:27 — with GitHub Actions Error
@VGau VGau temporarily deployed to docker-build-and-e2e December 6, 2024 12:29 — with GitHub Actions Inactive
@VGau VGau temporarily deployed to docker-build-and-e2e December 6, 2024 12:59 — with GitHub Actions Inactive
@VGau VGau temporarily deployed to docker-build-and-e2e December 9, 2024 08:25 — with GitHub Actions Inactive
@VGau VGau merged commit f58c12a into main Dec 9, 2024
26 checks passed
@VGau VGau deleted the feat/272-split-sdk-and-postman branch December 9, 2024 11:12
thedarkjester pushed a commit that referenced this pull request Dec 10, 2024
* feat: split postman and sdk

* fix: update postman client and sendMessage script

* fix: clean the sdk

* fix: update sdk dependencies

* fix: remove .env.sample file

* fix: remove testing helpers from the build

* fix: update gas provider in linea sdk and update postman client

* fix: update postman dependencies

* fix: update postman dockerfile and fix tests imports and types

* fix: remove unused code in the sdk + move typechain folder

* fix: remove unused code + fix imports in postman

* fix: pnpm lock file issue

* fix: import issue

* fix: case sensitive file issue

* fix: update sdk fees options and update exports

* fix: remove postman unused code and adjust imports and tests

* fix: update contracts abis + clean error parsing

* fix: update postman based on new SDk changes

* add readme + remove unused interface in postman

* fix: rename Base.ts file to BaseError.ts

* fix: rename Base.ts file to BaseError.ts in postman

* chore: update readme for the postman

* fix: rename maxFeePerGas to maxFeePerGasCap

* fix: update DefaultGasProvider fees check

* fix: default gas provider test issue

* fix: update main ci filter

* fix: issue in default gas provider
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.

3 participants