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

All tests passed with the new Lock file with only some more warnings #2643

Merged

Conversation

Sumit112192
Copy link
Contributor

📝 Description

Type: 🎢 infrastructure

I ran the tests using the new environment created using the new lock file. All tests were passed.
= 263 passed, 1579 skipped, 15 xfailed, 3 xpassed, 43 warnings, 13 errors in 44.20s =
When I ran the tests using the running/default environment,
= 263 passed, 1579 skipped, 15 xfailed, 3 xpassed, 19 warnings, 13 errors in 19.08s =

Warnings have increased from 19 to 43, and the run time has increased from 19s to 44s.

🚦 Testing

How did you test these changes?

  • Testing pipeline
  • Other method (describe)
  • My changes can't be tested (explain why)

☑️ Checklist

  • I requested two reviewers for this pull request
  • I updated the documentation according to my changes
  • I built the documentation by applying the build_docs label

Note: If you are not allowed to perform any of these actions, ping (@) a contributor.
@atharva-2001

@atharva-2001 atharva-2001 marked this pull request as ready for review June 4, 2024 12:39
@atharva-2001 atharva-2001 merged commit 55f209c into tardis-sn:update_lockfiles_2024 Jun 4, 2024
5 checks passed
wkerzendorf added a commit that referenced this pull request Jun 7, 2024
* Initial commit

* Update lockfiles and unpin qgrid

* Temporarily change tests.yml to use the new branch for lock files. To be undoed before merging.

* All tests passed with the new Lock file with only some more warnings (#2643)

* Update .github/workflows/tests.yml

---------

Co-authored-by: Sumit Gupta <[email protected]>
Co-authored-by: Wolfgang Kerzendorf <[email protected]>
@Sumit112192 Sumit112192 deleted the LockFileTest branch September 8, 2024 13:08
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