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

!refactor: rename GetSharesByNamespace to GetNamespaceData #3902

Merged
merged 4 commits into from
Nov 1, 2024

Conversation

cristaloleg
Copy link
Contributor

@cristaloleg cristaloleg commented Oct 30, 2024

Fixes #3892

@cristaloleg cristaloleg added kind:refactor Attached to refactoring PRs kind:chore labels Oct 30, 2024
@cristaloleg cristaloleg self-assigned this Oct 30, 2024
@cristaloleg cristaloleg force-pushed the chore/rename-GetSharesByNamespace branch from 171c28f to 8911d12 Compare November 1, 2024 15:53
@cristaloleg cristaloleg marked this pull request as ready for review November 1, 2024 15:54
@renaynay renaynay added the kind:break! Attached to breaking PRs label Nov 1, 2024
@Wondertan
Copy link
Member

attach the issue it closes pls

@renaynay
Copy link
Member

renaynay commented Nov 1, 2024

This PR is breaking, can you change title to reflec tthat?

@cristaloleg cristaloleg changed the title chore: rename GetSharesByNamespace to GetNamespaceData !refactor: rename GetSharesByNamespace to GetNamespaceData Nov 1, 2024
Copy link
Member

@renaynay renaynay left a comment

Choose a reason for hiding this comment

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

are there any swamp tests that are affected by this?

@cristaloleg cristaloleg merged commit c6062c6 into main Nov 1, 2024
34 checks passed
@cristaloleg cristaloleg deleted the chore/rename-GetSharesByNamespace branch November 1, 2024 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:break! Attached to breaking PRs kind:chore kind:refactor Attached to refactoring PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

share/shwap: Getter: GetSharesByNamespace -> GetNamespaceData
3 participants