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

Extend results schema to allow test phases #2933

Merged
merged 1 commit into from
May 31, 2024
Merged

Conversation

happz
Copy link
Collaborator

@happz happz commented May 15, 2024

Related to #2826.

Pull Request Checklist

  • implement the feature
  • write the documentation
  • update the specification

@happz happz added code | schema Schema used for validating config files area | results Related to how tmt stores and shares results labels May 15, 2024
@happz happz added this to the 1.34 milestone May 15, 2024
@happz
Copy link
Collaborator Author

happz commented May 15, 2024

Once teemtee/fmf#189 lands, it should be trivial to add schema validation tests.

@happz happz marked this pull request as ready for review May 15, 2024 13:40
@happz happz added the status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish. label May 30, 2024
@happz
Copy link
Collaborator Author

happz commented May 31, 2024

Specification-only change, merging without the full set of functional tests.

@happz happz merged commit 61b723b into main May 31, 2024
11 of 19 checks passed
@happz happz deleted the results-store-phases branch May 31, 2024 07:38
happz added a commit that referenced this pull request Jun 2, 2024
@seberm seberm mentioned this pull request Jul 23, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area | results Related to how tmt stores and shares results code | schema Schema used for validating config files status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants