Skip to content

Conversation

@Ranieri93
Copy link

  • Dependency Updates: All production and development dependencies have been updated to their latest versions to incorporate the latest features and security patches.
  • Test Fixes: Resolved a mock scoping issue that caused multiple tests to fail by moving hooks to the top-level scope.
  • Image Version Refactoring: Replaced hardcoded Docker image tags with a centralized, pinned-version constant.
  • Documentation: The documentation has been updated to reflect the new recommended, pinned Docker image versions.

Copy link
Owner

@alfonsograziano alfonsograziano left a comment

Choose a reason for hiding this comment

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

LGTM - thanks Ranieri, it's always great to get helpful contributions like this!

@alfonsograziano
Copy link
Owner

It looks like we have some tests failing, do you mind taking a look?

- Dependency Updates: All production and development dependencies have been
  updated to their latest versions to incorporate the latest features
  and security patches.
- Test Fixes: Resolved a mock scoping issue that caused multiple tests
  to fail by moving hooks to the top-level scope.
- Image Version Refactoring: Replaced hardcoded Docker image tags with a
  centralized, pinned-version constant.
- Documentation: The documentation has been updated to reflect the new
  recommended, pinned Docker image versions.
@Ranieri93 Ranieri93 force-pushed the update_packages_and_fix_tests branch from cb35b93 to 68e5d2a Compare August 21, 2025 06:33
@Ranieri93
Copy link
Author

Hey! It seems that runJsEphemeral screenshot with Playwright is flaky.
The consistent way in which I was able to make it pass is by always deleting all leftover containers before re-running it.
I increased the timeout threshold; let's see if the problem arises anyway!

@alfonsograziano alfonsograziano merged commit d3dc621 into alfonsograziano:master Aug 21, 2025
2 of 3 checks 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