-
Notifications
You must be signed in to change notification settings - Fork 13
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: use generics in pagination #417
Conversation
e177ac2
to
2987cf3
Compare
timeout-minutes: 20 | ||
steps: | ||
- name: Install Go | ||
run: apk add go |
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.
no need to install go - 1.18 is already installed as part of the docker image.
Before it would install go1.17.
- name: Install Go | ||
run: apk add go | ||
- name: Install Terraform | ||
run: apk add terraform |
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.
installing terraform instead...
- name: Checkout code | ||
uses: actions/checkout@v2 | ||
uses: actions/checkout@v3 |
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.
noticed some depreciation warnings. Went away when switching to v3.
type Paginated interface { | ||
// FIXME: Environment | ||
Environment |
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.
Adding back the generics in this file...
In case more paging is added in the future to other API calls. Generics will make it very simple to add support for.
2987cf3
to
19ff58a
Compare
Issue & Steps to Reproduce / Feature Request
closes #415
Solution
Harness was running in go1.17 because the terraform docker image was using golang 1.17.
Switched to using a go1.18 docker image and installed terraform via
apk
.Making this change allows me to use generics (A new feature in go1.18).