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

feat(chore): adapt to storages usage of protobuf #538

Merged
merged 5 commits into from
Aug 14, 2024

Conversation

mohammed90
Copy link
Contributor

Depends on darkweak/storages#8 and requires amendment after merging the storages PR.

I noticed the tyk package still depends on the older API of Souin which has storages within Souin repo. I left it as-is, though it'll require minor changes to work. I can do it if it was missed from before.

Copy link

netlify bot commented Jul 20, 2024

Deploy Preview for teal-sprinkles-4c7f14 canceled.

Name Link
🔨 Latest commit 3e6dfc4
🔍 Latest deploy log https://app.netlify.com/sites/teal-sprinkles-4c7f14/deploys/66ba82da6a48860008eba6f1

@mohammed90
Copy link
Contributor Author

I tried making the tests pass with replacements. It doesn't work 😅 I guess I have to wait for the other PR.

Signed-off-by: Mohammed Al Sahaf <msaa1990@gmail.com>
@darkweak darkweak force-pushed the gob-to-protobuf branch 2 times, most recently from 78a234d to c5c126e Compare August 12, 2024 21:41
@mohammed90 mohammed90 marked this pull request as ready for review August 14, 2024 05:58
@darkweak darkweak merged commit 295815c into darkweak:master Aug 14, 2024
19 of 20 checks passed
@mohammed90 mohammed90 deleted the gob-to-protobuf branch August 14, 2024 06:46
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