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

fix: conversion of hex to decimal (#1167) #1169

Merged
merged 1 commit into from
Dec 11, 2023
Merged

Conversation

Yashk767
Copy link
Contributor

@Yashk767 Yashk767 commented Dec 1, 2023

  • fix: conversion of hex to decimal

  • fix: convertToNumber tests fixed

Description

Merging develop to releases/v1.1.0 with the hex conversion fix that was earlier merged to develop

* fix: conversion of hex to decimal

* fix: convertToNumber tests fixed
@SkandaBhat SkandaBhat merged commit 703aec5 into releases/v1.1.0 Dec 11, 2023
2 checks passed
Yashk767 added a commit that referenced this pull request Jan 15, 2024
* fix: conversion of hex to decimal

* fix: convertToNumber tests fixed
Yashk767 added a commit that referenced this pull request Jan 17, 2024
#1177)

* fix: conversion of hex to decimal (#1167)

* fix: conversion of hex to decimal

* fix: convertToNumber tests fixed

* feat: added support for job returning results in hex array (#1174)

* feat: added support for jobs returning result in hex array

* feat: added tests for hex array decoding and regex functions

* refactor: fixed golangci-lint errors

* refactor: used hash of url and body as a cache key (#1176)

* refactor: used hash of url and body as cache key instead of just url

* fix: replaced with new cacheKey while reading from cache

* refactor: added tests for different job cases

* refcator: added missing coverage test cases

* refcator: fixed tests

* refactor: fixted tests by replcing binance API with coinbase API

* refactor: job index changed in test name

* fix: conversion of hex to decimal (#1167) (#1169) (#1178)

* fix: conversion of hex to decimal

* fix: convertToNumber tests fixed
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.

4 participants