Skip to content

Commit

Permalink
Drop Resty as a dependency (#576)
Browse files Browse the repository at this point in the history
* Unit tests passing

* Remove all references to Resty

* Fixed some integration tests

* Fixed failing integration tests

* Fix lint

* Fixed request body not being logged

* Fixed issue where request body did not get reset for retry

* Fix lint

* Fixed issue with image uploads

* build(deps): bump golang.org/x/text from 0.18.0 to 0.19.0 (#587)

* build(deps): bump golang.org/x/text from 0.18.0 to 0.19.0

Bumps [golang.org/x/text](https://github.com/golang/text) from 0.18.0 to 0.19.0.
- [Release notes](https://github.com/golang/text/releases)
- [Commits](golang/text@v0.18.0...v0.19.0)

---
updated-dependencies:
- dependency-name: golang.org/x/text
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* Ran make tidy

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: ezilber-akamai <ezilber@akamai.com>

* ci: Fix workflow failures in PRs due to slack notify step (#588)

* allow some vars to be shared

* add target branch

* remove target branch

* revert to push_request and only run slack notify when push to main

* Fixed intermittent error

* Addressed more PR comments

* Addressed more PR comments

* Addressed some more PR comments

* Added default values for retry times

* Updated body type in requestParams

* Addressed more PR comments

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Youjung Kim <126618609+ykim-1@users.noreply.github.com>
  • Loading branch information
3 people authored Oct 25, 2024
1 parent 2b51d5c commit 4a2a1e6
Show file tree
Hide file tree
Showing 26 changed files with 691 additions and 1,095 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ jobs:

- name: Fail if changes
run: git diff-index --exit-code HEAD

test:
runs-on: ubuntu-latest
env:
Expand Down Expand Up @@ -81,7 +81,7 @@ jobs:
notify-slack:
runs-on: ubuntu-latest
needs: [test]
if: always() && github.repository == 'linode/linodego' # Run even if integration tests fail and only on main repository
if: always() && github.ref == 'refs/heads/main' && github.event_name == 'push' # Run even if integration tests fail and only on main repository

steps:
- name: Notify Slack
Expand Down
Loading

0 comments on commit 4a2a1e6

Please sign in to comment.