Skip to content

Conversation

@JPonsa
Copy link
Contributor

@JPonsa JPonsa commented Mar 6, 2024

add environmental variable ''DPS_CACHEBOOL' to activate or deactivate cache. By default True.

add environmental variable ''DPS_CACHEBOOL' to activate or deactivate cache. By default True.
@JPonsa
Copy link
Contributor Author

JPonsa commented Mar 6, 2024

However, it would be cleaner to have config option to disable cache per request

@insop
Copy link
Contributor

insop commented Mar 7, 2024

Thank you very much, I needed this, too.

@insop
Copy link
Contributor

insop commented Mar 9, 2024

@okhat

@JPonsa
Copy link
Contributor Author

JPonsa commented Mar 16, 2024

@insop is there any action from my side?

@okhat
Copy link
Collaborator

okhat commented Mar 16, 2024

Not sure why the checks had failed. Rerunning them and will merge if all is good.

Update: OK they're failing again looks like....

@okhat
Copy link
Collaborator

okhat commented Mar 16, 2024

@isaacbmiller do you happen to know if this failure is expected?

@isaacbmiller
Copy link
Collaborator

@JPonsa merge main back into your branch and then the checks will pass

@okhat if PRs were created in the ~1 week period where I broke CI from forks, then the tests will fail

@isaacbmiller
Copy link
Collaborator

CI working as intended

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.

4 participants