jetstack / navigator

Managed Database-as-a-Service (DBaaS) on Kubernetes
Apache License 2.0
271 stars 31 forks source link

Remove duplicate Int32Ptr and Int64Ptr functions #296

Closed wallrj closed 6 years ago

wallrj commented 6 years ago

In https://github.com/jetstack/navigator/pull/256/files#r174153467 @munnerz wrote:

Maybe it's time to create test/util/pointer to expose these common pointer-related functions for re-use. We've written this function so many times now

Release note:

NONE
munnerz commented 6 years ago

/lgtm /approve

jetstack-bot commented 6 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: munnerz

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/jetstack/navigator/blob/master/OWNERS)~~ [munnerz] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment