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.

Sign up for GitHub

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

Jump to bottom

Remove data-path field from tmt-report-result output #3270

Merged
merged 2 commits into from
Oct 10, 2024
Merged

Remove data-path field from tmt-report-result output #3270

merged 2 commits into from
Oct 10, 2024

Conversation

seberm
Copy link
Collaborator

@seberm seberm commented Oct 8, 2024 •
edited by psss
Loading

Related to:

Pull Request Checklist

  • implement the feature

@seberm seberm added area | results Related to how tmt stores and shares results code | trivial A simple patch - a couple of lines, an easy-to-understand change, a typo fix. ci | full test Pull request is ready for the full test execution labels Oct 8, 2024
@seberm seberm added this to the 1.38 milestone Oct 8, 2024
@seberm seberm mentioned this pull request Oct 8, 2024
Merged
16 tasks
psss
psss approved these changes Oct 10, 2024
View reviewed changes
Copy link
Collaborator

@psss psss left a comment

Choose a reason for hiding this comment

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

Thanks!

@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 Oct 10, 2024
@psss psss force-pushed the feature/remove-data-path-for-tmt-report-result branch from 9cf3ac3 to a465458 Compare October 10, 2024 12:41
@psss psss changed the title Remove data-path field from tmt-report-result output Remove data-path field from tmt-report-result output Oct 10, 2024
@psss psss self-assigned this Oct 10, 2024
@psss psss merged commit 3f51582 into main Oct 10, 2024
22 of 23 checks passed
@psss psss deleted the feature/remove-data-path-for-tmt-report-result branch October 10, 2024 19:56
The-Mule pushed a commit to The-Mule/tmt that referenced this pull request Oct 14, 2024
@seberm @The-Mule
Remove data-path field from tmt-report-result output (teemtee#3270)
8f53ec2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@happz happz happz approved these changes

@psss psss psss approved these changes

@lukaszachy lukaszachy Awaiting requested review from lukaszachy lukaszachy is a code owner

@thrix thrix Awaiting requested review from thrix thrix is a code owner

@janhavlin janhavlin Awaiting requested review from janhavlin janhavlin is a code owner

Assignees

@psss psss

Labels
area | results Related to how tmt stores and shares results ci | full test Pull request is ready for the full test execution code | trivial A simple patch - a couple of lines, an easy-to-understand change, a typo fix. status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish.
Projects
None yet
Milestone
1.38
Development

Successfully merging this pull request may close these issues.
3 participants
@seberm @happz @psss