Skip to content

Conversation

@Y4vra
Copy link

@Y4vra Y4vra commented May 3, 2025

No description provided.

@Y4vra Y4vra added the documentation 📝 Improvements or additions to documentation label May 3, 2025
@Y4vra Y4vra self-assigned this May 3, 2025
Copy link

@VicenteMG04 VicenteMG04 left a comment

Choose a reason for hiding this comment

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

Great job. I just found a minor format issue that is worth solving at minimum effort. Thank you so much!


Tests are executed using *Jest* as the runner, configured via `jest_config.js`. The `test_environment_setup` script ensures that all required services (e.g., gatewayservice, gameservice, questionservice...) are initialized and cleaned before and after test runs.

Our E2E workflows include:

Choose a reason for hiding this comment

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

The mentioned E2E workflows appear with a strange format. This happens because you are trying to create a list/enummeration inside a table row. Try looking for a better format for this!

@Y4vra Y4vra requested a review from VicenteMG04 May 3, 2025 18:39
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 3, 2025

Copy link

@VicenteMG04 VicenteMG04 left a comment

Choose a reason for hiding this comment

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

All good! Great job, mate! 👍

@VicenteMG04 VicenteMG04 merged commit 20d64b7 into master May 3, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation 📝 Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants