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

Inventory plugin: drop api_token in favor of oauth_token #301

Merged
merged 2 commits into from
Aug 12, 2023
Merged

Conversation

mamercad
Copy link
Collaborator

@mamercad mamercad commented Feb 1, 2023

SUMMARY

Fixes #300; drop api_token in favor of oauth_token.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME
  • digitalocean inventory plugin
ADDITIONAL INFORMATION
N/A

@mamercad mamercad changed the title Drop api_token in favor of oauth_token Inventory plugin: drop api_token in favor of oauth_token Feb 1, 2023
@codecov
Copy link

codecov bot commented Feb 1, 2023

Codecov Report

Merging #301 (e5a320b) into main (d4ba6a5) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #301      +/-   ##
==========================================
- Coverage   31.10%   31.08%   -0.02%     
==========================================
  Files          45       45              
  Lines        3588     3587       -1     
  Branches      732      732              
==========================================
- Hits         1116     1115       -1     
  Misses       2452     2452              
  Partials       20       20              
Flag Coverage Δ
sanity 26.40% <0.00%> (-0.03%) ⬇️
units 64.49% <100.00%> (-0.09%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
plugins/inventory/digitalocean.py 70.83% <100.00%> (-0.31%) ⬇️

@mamercad mamercad merged commit 166b5d0 into main Aug 12, 2023
9 checks passed
@mamercad mamercad deleted the issue-300 branch August 12, 2023 02:49
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.

inventory plugin declares options api_token / oauth_token twice
1 participant