Skip to content

Fix/pubsub migrations #1842

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

Merged
merged 47 commits into from
Jun 27, 2025
Merged

Fix/pubsub migrations #1842

merged 47 commits into from
Jun 27, 2025

Conversation

Umang01-hash
Copy link
Member

@Umang01-hash Umang01-hash commented Jun 12, 2025

Pull Request Template

Description:

Screenshot 2025-06-12 at 12 35 02 PM

Checklist:

  • I have formatted my code using goimport and golangci-lint.
  • All new code is covered by unit tests.
  • This PR does not decrease the overall code coverage.
  • I have reviewed the code comments and documentation for clarity.

Thank you for your contribution!

Copy link
Member

@coolwednesday coolwednesday left a comment

Choose a reason for hiding this comment

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

Please remove this unnecessary file. Also we can add in migrations docs an example of specifically the Query method and its usecase & what arguments it takes.

aryanmehrotra
aryanmehrotra previously approved these changes Jun 26, 2025
@Umang01-hash Umang01-hash merged commit bf94f17 into development Jun 27, 2025
20 checks passed
@Umang01-hash Umang01-hash deleted the fix/pubsub_migrations branch June 27, 2025 09:26
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