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

RSpec quarantining support #484

Merged

Conversation

gnalh
Copy link
Collaborator

@gnalh gnalh commented Mar 18, 2025

No description provided.

Copy link

trunk-io bot commented Mar 18, 2025

😎 Merged successfully - details.

@gnalh gnalh force-pushed the gabe/trunk-14369-support-quarantining-within-rspec-plugin branch from 4718fef to 04817c9 Compare March 18, 2025 02:37
Copy link

trunk-staging-io bot commented Mar 18, 2025

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

Copy link

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

1 similar comment
Copy link

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

@gnalh gnalh force-pushed the gabe/trunk-14369-support-quarantining-within-rspec-plugin branch 4 times, most recently from dad5cce to 5c6b9ba Compare March 19, 2025 19:42
@gnalh gnalh changed the base branch from main to gabe/add-list-quarantine-endpoint March 19, 2025 19:43
@gnalh gnalh force-pushed the gabe/trunk-14369-support-quarantining-within-rspec-plugin branch from 5c6b9ba to ac5789a Compare March 19, 2025 22:02
@gnalh gnalh changed the base branch from gabe/add-list-quarantine-endpoint to main March 19, 2025 22:02
@gnalh gnalh force-pushed the gabe/trunk-14369-support-quarantining-within-rspec-plugin branch 4 times, most recently from 3c3380e to 7903788 Compare March 26, 2025 19:58
@@ -66,9 +66,7 @@ pub struct TelemetryUploadMetricsRequest {

#[derive(Debug, Serialize, Clone, Deserialize, PartialEq)]
pub struct PageQuery {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The api updated the names before release.

@gnalh gnalh force-pushed the gabe/trunk-14369-support-quarantining-within-rspec-plugin branch from 7903788 to 6fd6090 Compare March 26, 2025 20:56
@gnalh gnalh force-pushed the gabe/trunk-14369-support-quarantining-within-rspec-plugin branch 3 times, most recently from f3da1c2 to 6e8419e Compare March 26, 2025 21:47
@@ -230,7 +342,6 @@ impl MutTestReport {
Status::Skipped => test.status = TestCaseRunStatus::Skipped.into(),
Status::Unspecified => test.status = TestCaseRunStatus::Unspecified.into(),
}
// test.status = status;
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is extraneous, we set it above.

@gnalh gnalh force-pushed the gabe/trunk-14369-support-quarantining-within-rspec-plugin branch from 6e8419e to f23d8c5 Compare March 26, 2025 22:02
@gnalh gnalh marked this pull request as ready for review March 26, 2025 22:26
@gnalh gnalh requested a review from cmillar-trunk as a code owner March 26, 2025 22:26
Copy link

trunk-io bot commented Mar 26, 2025

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

@trunk-io trunk-io bot merged commit 9c4f4c1 into main Mar 27, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants