Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

feat: Always use BigInt #321

Merged
merged 1 commit into from
Jun 7, 2022
Merged

feat: Always use BigInt #321

merged 1 commit into from
Jun 7, 2022

Conversation

yevgenypats
Copy link
Member

🎉 Thank you for making CloudQuery awesome by submitting a PR 🎉

Summary

This is a pre-step to supporting only TypeBigInt. we don't need necessarly to remove this as this will break a bit of backward compatibility but at least right now we shouldn't have any int mismatch errors as we always use BigInt.


Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines 🧑‍🎓
  • Run go fmt to format your code 🖊
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests 🧪
  • Ensure the status checks below are successful ✅

@yevgenypats yevgenypats requested a review from a team as a code owner June 7, 2022 10:42
@yevgenypats yevgenypats requested review from shimonp21 and removed request for a team June 7, 2022 10:42
@yevgenypats
Copy link
Member Author

Also, found a small bug where we didn't support *uint16

Copy link
Member

@disq disq left a comment

Choose a reason for hiding this comment

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

LGTM

@yevgenypats yevgenypats merged commit 2033349 into main Jun 7, 2022
@yevgenypats yevgenypats deleted the feat/always_use_big_int branch June 7, 2022 10:49
kodiakhq bot pushed a commit that referenced this pull request Jun 7, 2022
🤖 I have created a release *beep* *boop*
---


## [0.10.10](v0.10.9...v0.10.10) (2022-06-07)


### Features

* Always use BigInt ([#321](#321)) ([2033349](2033349))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants