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

chore: Random ids rework part3 #2833

Merged
merged 14 commits into from
May 27, 2024
Merged

chore: Random ids rework part3 #2833

merged 14 commits into from
May 27, 2024

Conversation

sfc-gh-asawicki
Copy link
Collaborator

@sfc-gh-asawicki sfc-gh-asawicki commented May 23, 2024

Continuation of ids rework:

  • Do not use random.String() for the identifiers
  • Do not use random.StringN() for the identifiers
  • Do not use random.UUID() for the identifiers
  • Do not use random.AlphanumericN() for the identifiers
  • Do not use random.Alpha() for the identifiers
  • Do not use random.StringRange() for the identifiers

Copy link

Integration tests failure for 94a432c6cb689b9f82048ac2ef7c003c8127cdf8

@sfc-gh-asawicki sfc-gh-asawicki force-pushed the random-ids-rework-part3 branch from 94a432c to 2d78f54 Compare May 24, 2024 11:57
Copy link

Integration tests failure for 2d78f548da77c4d528dcdaa6775790b41daa42d2

Copy link

Integration tests success for 2d78f548da77c4d528dcdaa6775790b41daa42d2

@sfc-gh-asawicki sfc-gh-asawicki marked this pull request as ready for review May 27, 2024 08:17
@sfc-gh-asawicki sfc-gh-asawicki merged commit 36ead85 into main May 27, 2024
10 checks passed
@sfc-gh-asawicki sfc-gh-asawicki deleted the random-ids-rework-part3 branch May 27, 2024 11:21
sfc-gh-jcieslak pushed a commit that referenced this pull request Jun 6, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.92.0](v0.91.0...v0.92.0)
(2024-06-06)


### 🎉 **What's new:**

* Add Api Authentication security integration to sdk
([#2840](#2840))
([57a07ee](57a07ee))
* Add External Oauth security integration to sdk
([#2835](#2835))
([82d1c09](82d1c09))
* add network rules
([#2746](#2746))
([c79fa29](c79fa29))
* Add SCIM and SAML2 security integrations to sdk
([#2799](#2799))
([1312ff1](1312ff1))
* Add Snowflake Oauth security integration to sdk
([#2830](#2830))
([b576f29](b576f29))
* Database resource v1 readiness
([#2834](#2834))
([30fe136](30fe136))
* Database SDK upgrade
([#2814](#2814))
([750fe37](750fe37))


### 🔧 **Misc**

* accept non-pointer values in the generated builder methods
([#2816](#2816))
([c29fbf1](c29fbf1))
* Add a script for creating labels
([#2778](#2778))
([ce0fbad](ce0fbad))
* Adjust before 0.92.0
([#2857](#2857))
([0598656](0598656))
* Continue random ids rework
([#2819](#2819))
([f20940c](f20940c))
* Random ids rework part3
([#2833](#2833))
([36ead85](36ead85))
* Random ids rework part4
([#2837](#2837))
([64518a3](64518a3))
* Update issue for table and warehouse redesign state
([#2845](#2845))
([149e55e](149e55e))


### 🐛 **Bug fixes:**

* Fix failing integration tests
([#2832](#2832))
([2e2ca6c](2e2ca6c))
* Fix QUOTED_IDENTIFIERS_IGNORE_CASE parameter test
([#2841](#2841))
([92ad1d3](92ad1d3))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: snowflake-release-please[bot] <105954990+snowflake-release-please[bot]@users.noreply.github.com>
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