Skip to content

wrote tests for slurm/connection.py#99

Merged
joshfactorial merged 2 commits into
llmflux_using_apifrom
api_add_tests
Jun 18, 2026
Merged

wrote tests for slurm/connection.py#99
joshfactorial merged 2 commits into
llmflux_using_apifrom
api_add_tests

Conversation

@joshfactorial

Copy link
Copy Markdown
Collaborator

No description provided.

@joshfactorial joshfactorial requested a review from Ralhazmy1 May 21, 2026 15:40
@joshfactorial

Copy link
Copy Markdown
Collaborator Author

@Ralhazmy1 Adding some tests

@gitguardian

gitguardian Bot commented May 21, 2026

Copy link
Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
33056202 Triggered Generic High Entropy Secret f80f170 tests/slurm/test_connection.py View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@joshfactorial

Copy link
Copy Markdown
Collaborator Author

gitguardian incorrectly flagging a test value

@joshfactorial joshfactorial merged commit 2571133 into llmflux_using_api Jun 18, 2026
1 check passed
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