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

Fix apierr.APIError pointer receivers #307

Merged
merged 1 commit into from
Feb 17, 2023
Merged

Fix apierr.APIError pointer receivers #307

merged 1 commit into from
Feb 17, 2023

Conversation

nfx
Copy link
Contributor

@nfx nfx commented Feb 17, 2023

Follow-up work for #298

@codecov-commenter
Copy link

Codecov Report

Base: 40.87% // Head: 40.87% // No change to project coverage 👍

Coverage data is based on head (8924a61) compared to base (824895b).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #307   +/-   ##
=======================================
  Coverage   40.87%   40.87%           
=======================================
  Files          46       46           
  Lines        2662     2662           
=======================================
  Hits         1088     1088           
  Misses       1485     1485           
  Partials       89       89           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@nfx nfx merged commit a0b269d into main Feb 17, 2023
@nfx nfx deleted the fix/err-type branch February 17, 2023 10:36
nfx added a commit that referenced this pull request Feb 17, 2023
 * Change APIError to use pointer receiver ([#298](#298)).
 * Drop duplicate prefix in randomized email ([#299](#299)).
 * Expand semver pattern to be compliant with https://semver.org ([#302](#302)).
 * Fix `apierr.APIError` pointer receivers ([#307](#307)).
 * Skip loading config if auth is already explicitly configured directly ([#306](#306)).
 * Sync fixes for smallest node selection from Terraform ([#301](#301)).
 * Updated from OpenAPI spec ([#305](#305)).

Dependency updates:

 * Bump google.golang.org/api from 0.109.0 to 0.110.0 ([#303](#303)).
@nfx nfx mentioned this pull request Feb 17, 2023
nfx added a commit that referenced this pull request Feb 17, 2023
* Change APIError to use pointer receiver
([#298](#298)).
* Drop duplicate prefix in randomized email
([#299](#299)).
* Expand semver pattern to be compliant with https://semver.org
([#302](#302)).
* Fix `apierr.APIError` pointer receivers
([#307](#307)).
* Skip loading config if auth is already explicitly configured directly
([#306](#306)).
* Sync fixes for smallest node selection from Terraform
([#301](#301)).
* Updated from OpenAPI spec
([#305](#305)).

Dependency updates:

* Bump google.golang.org/api from 0.109.0 to 0.110.0
([#303](#303)).
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.

2 participants