Skip to content

use_multi_tenancy_endpoints_II: tests running#208

Merged
DanielElisenberg merged 3 commits intomainfrom
use_multi_tenancy_endpoints_II
Nov 3, 2025
Merged

use_multi_tenancy_endpoints_II: tests running#208
DanielElisenberg merged 3 commits intomainfrom
use_multi_tenancy_endpoints_II

Conversation

@DanielElisenberg
Copy link
Collaborator

No description provided.

@DanielElisenberg DanielElisenberg requested a review from a team as a code owner October 30, 2025 06:46
Copy link
Contributor

@linnbjornerud linnbjornerud left a comment

Choose a reason for hiding this comment

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

Overall this looks good to me! But see my comment about the snake/camel issue.

linnbjornerud
linnbjornerud previously approved these changes Nov 3, 2025
Copy link
Contributor

@linnbjornerud linnbjornerud left a comment

Choose a reason for hiding this comment

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

Looks good 👍🏻

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 3, 2025

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@DanielElisenberg
Copy link
Collaborator Author

@linnbjornerud re-requested review due to format errors. I'll fix my broken pre-commit today 🙏🏻

@DanielElisenberg DanielElisenberg merged commit 1e0632b into main Nov 3, 2025
5 checks passed
@DanielElisenberg DanielElisenberg deleted the use_multi_tenancy_endpoints_II branch November 3, 2025 07:56
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