Skip to content

Conversation

@johnwalz97
Copy link
Contributor

Pull Request Description

What and why?

How to test

What needs special review?

Dependencies, breaking changes, and deployment notes

Release notes

Checklist

  • What and why
  • Screenshots or videos (Frontend)
  • How to test
  • What needs special review
  • Dependencies, breaking changes, and deployment notes
  • Labels applied
  • PR linked to Shortcut
  • Unit tests added (Backend)
  • Tested locally
  • Documentation updated (if required)
  • Environment variable additions/changes documented (if required)

actions-user and others added 22 commits July 24, 2025 18:20
Merge Main into Prod (prod-2.8.28)
Merge Main into Prod (prod-2.8.29)
Merge Main into Prod (prod-2.9.1)
…heck

Hotfix: Remove LLM is_configured() Check from Test Description Generation
Merge Main into Prod (prod-2.9.5)
…ata-decorators

Hotfix/john/dont require metadata decorators
@johnwalz97 johnwalz97 requested a review from nibalizer October 23, 2025 16:11
@github-actions
Copy link
Contributor

Pull requests must include at least one of the required labels: internal (no release notes required), highlight, enhancement, bug, deprecation, documentation. Except for internal, pull requests must also include a description in the release notes section.

@github-actions
Copy link
Contributor

PR Summary

This PR enhances the test loading mechanism by ensuring that every test function has defined __tags__ and __tasks__ attributes. If these attributes are absent, they are automatically initialized to empty lists. This change ensures consistency in how test metadata is handled and prevents potential errors when these attributes are accessed later in the testing workflow.

Test Suggestions

  • Write tests that deliberately define test functions without __tags__ and __tasks__ to verify they are automatically set to empty lists.
  • Validate that if a test function already has __tags__ or __tasks__ set, the existing values are preserved and not overwritten.
  • Check the integration with the test registry to confirm that the attributes are correctly attached to the test functions after registration.

@johnwalz97 johnwalz97 added the internal Not to be externalized in the release notes label Oct 23, 2025
@johnwalz97 johnwalz97 merged commit 2a8e956 into main Oct 23, 2025
14 of 17 checks passed
@johnwalz97 johnwalz97 deleted the prod-to-main branch October 23, 2025 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Not to be externalized in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants