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

💡 [patch] Repo maintenance: deps upgrades #92

Merged
merged 3 commits into from
Aug 6, 2024
Merged

Conversation

ronjouch
Copy link
Contributor

@ronjouch ronjouch commented Aug 6, 2024

The usual:

  1. Relock
  2. Major upgrade, see individual commits for proof migration guides were followed

How was this tested?

Tests & linter pass

@@ -60,7 +60,6 @@ export class RedisCache extends CacheInstance {
// This prevents get/setValue calls from hanging if there is no active connection
enableOfflineQueue: false,
});
this.redisClient.scan
Copy link
Contributor Author

@ronjouch ronjouch Aug 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Must have been a cat walking on keyboard 😉 . Thankfully, was not doing anything.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😳 close call

@ronjouch ronjouch changed the title Repo maintenance 💡 [patch] Repo maintenance: deps upgrades Aug 6, 2024
@@ -0,0 +1,29 @@
import eslint from '@eslint/js';
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reminder to reviewers: Cachette is public, no revealing internal sikrit world domination plans here in comments. 🙏

@ronjouch ronjouch merged commit dae385c into master Aug 6, 2024
4 checks passed
@ronjouch ronjouch deleted the repo-maintenance branch August 6, 2024 18:47
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.

3 participants