-
Notifications
You must be signed in to change notification settings - Fork 19
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
(CORE) Add error code and catch error #1132
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
martinkersner
approved these changes
Feb 19, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
KelvinThai
approved these changes
Feb 19, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
nick-bisonai
added a commit
that referenced
this pull request
Feb 21, 2024
* feat: init feat: add workspace fix: remove ignore file feat: implement `SignTxAsFeePayer` fix: go mod tidy test: add test codes. feat: add missing implementations fix: update package name, fix queries test: updates during test code implementation test: add test codes. feat: add missing implementations fix: update package name, fix queries feat: add missing implementations fix: update package name, fix queries test: update test codes, fix minor errors test: update test codes, fix minor errors test: update for test fix: remove logs test: add contract reading codes test: wip feat: add missing implementations fix: update package name, fix queries test: update test codes, fix minor errors test: update test codes, fix minor errors test: update for test fix: remove logs test: add contract reading codes test: wip refactor: update names and cleanup test: wip feat: add dockerfile and github workflow feat: update github action files * fix: go mod tidy * fix: go mod tidy, update project path * feat: local docker compose update * feat: update google secret manager implementation * fix: remove test purpose logging * fix: update github action * test: tmp file to trigger GA * test: update github action * fix: update migration file * fix: fix syntax error * fix: rollback * fix: rollback feat: update fp-pk handling code wip * fix: trim 0x for pk * fix: remove comment and logs * fix: handle 0x prefix for pk * fix: update env var handler * fix: update context handling * fix: remove unnecessary comment * fix: update based on feedbacks * fix: typo * fix: remove underscored variables * feat: example k6 script (CORE) Add error code and catch error (#1132) * feat: add error code and catch error * test: trigger ga * fix: Modify location of jest binary * fix: remove tmp file --------- Co-authored-by: jay-bisonai <[email protected]> (OraklNode) Basic API (#1134) * feat: init * fix: remove comments * fix: go mod tidy fix: fix linter err (#1144) (OraklNode) Update packages, add test codes (#1133) * test: add tests * feat: update singleton functionality * fix: mutex instead of sync.once * feat: take context as param * fix: update test, rollback singleton * fix: use sync.once * fix: rollback to use sync.Once, update tests * fix: teardown only once from main * fix: go mod tidy (OraklNode) Update testcodes & fix migration file (#1145) * feat: update testcodes, fix migration file * fix: update based on feedback Restructure foundry contract directory (#1135) * restructure contracts directory * fixed git action error for contracts * push to trigger github action * Update vrf/README.md Co-authored-by: Martin Kersner <[email protected]> * moved github action in contract-v0.2 to root repo * restructure contracts folders * fixed solhint error * fixed linting error * Push to trigger test action * removed comment for test action * modified deploy script and docker files follow new contract structure * updated path in aggregator deployment --------- Co-authored-by: Martin Kersner <[email protected]> fix: rm
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
catches econn refused
Type of change
Please delete options that are not relevant.
Checklist before requesting a review
Deployment