Skip to content

Conversation

@PossessedC0bra
Copy link
Member

@PossessedC0bra PossessedC0bra commented Dec 18, 2025

  • fix swapped arguments in call to create_tested for database connection
  • remove unnecessary cli parameters

Copilot AI review requested due to automatic review settings December 18, 2025 15:16
@PossessedC0bra PossessedC0bra changed the title fix incorrect call to create_tested of a database connection fix tenant.create command Dec 18, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a bug where arguments to create_tested were passed in the wrong order. The function signature expects Label.t as the first parameter and Url.t as the second parameter, but the call was incorrectly passing url first and label second.

Key Changes:

  • Corrected the argument order in the call to create_tested from create_tested url label to create_tested label url

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@PossessedC0bra PossessedC0bra force-pushed the fix-db-connection-create-tested-args branch from 8f47977 to 78c6b0b Compare December 19, 2025 09:41
@PossessedC0bra PossessedC0bra force-pushed the fix-db-connection-create-tested-args branch from 78c6b0b to 1c111c9 Compare December 19, 2025 14:05
@PossessedC0bra PossessedC0bra requested a review from mabiede January 6, 2026 09:14
@mabiede mabiede merged commit 95a2b73 into main Jan 6, 2026
7 checks passed
@mabiede mabiede deleted the fix-db-connection-create-tested-args branch January 6, 2026 09:59
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