Automattic / vip-cli

The VIP-CLI

https://docs.wpvip.com/vip-cli/

MIT License

58 stars 17 forks source link

Remove Kubernetes-specific checks for job existence in SQL import status #2014

Closed luismulinari closed 2 months ago

luismulinari commented 2 months ago

Description

Remove Kubernetes-specific checks for job existence in SQL import status.

Removed the check for isK8sResident when determining if jobs exist in the environment.
Simplified the condition to check for job existence regardless of Kubernetes residency.

Pull request checklist

[x] Update SETUP.md with any new environmental variables.
[x] Update the documentation.
[x] Manually test the relevant changes.
[x] Follow the pull request checklist
[x] Add/update automated tests as needed.

New release checklist

[x] Automated tests pass.
[x] The Preparing for release checklist is completed.

Steps to Test

Outline the steps to test and verify the PR here.

Example:

Check out PR.
Run npm run build
Run ./dist/bin/vip-cookies.js nom
Verify cookies are delicious.

github-actions[bot] commented 2 months ago

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

Package	Version	Score	Details

Scanned Manifest Files

sonarcloud[bot] commented 2 months ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud