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 caching and add process command #158

Merged
merged 1 commit into from
Nov 23, 2021
Merged

Fix caching and add process command #158

merged 1 commit into from
Nov 23, 2021

Conversation

synfinatic
Copy link
Owner

  • Local cache was not saving during auto-refresh causing
    future runs to not use the cache and slowing execution after 24hrs.
  • Add support for AWS credential_process output in ~/.aws/config
    files

Refs #157

@codecov
Copy link

codecov bot commented Nov 23, 2021

Codecov Report

Merging #158 (6859f25) into main (2153609) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #158   +/-   ##
=======================================
  Coverage   37.38%   37.38%           
=======================================
  Files          10       10           
  Lines        1471     1471           
=======================================
  Hits          550      550           
  Misses        880      880           
  Partials       41       41           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2153609...6859f25. Read the comment docs.

@synfinatic synfinatic changed the title Fix caching and add eval --process Fix caching and add process command Nov 23, 2021
 * Local cache was not saving during auto-refresh causing
    future runs to not use the cache and slowing execution after 24hrs.
 * Add support for AWS `credential_process` output in ~/.aws/config
   files

Refs #157
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.

1 participant