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

fix: stop running the malicious integration test during short mode #2163

Merged
merged 1 commit into from
Jul 25, 2023

Conversation

evan-forbes
Copy link
Member

Overview

the testnode is still hitting the race detector #1369, so we aren't running those integration tests with the race detector

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@evan-forbes evan-forbes added the CI item that directly relates to or modify the CI label Jul 25, 2023
@evan-forbes evan-forbes self-assigned this Jul 25, 2023
@MSevey MSevey requested a review from a team July 25, 2023 13:14
@evan-forbes evan-forbes merged commit b269f6e into main Jul 25, 2023
25 of 27 checks passed
@evan-forbes evan-forbes deleted the evan/stop-running-mal-node-in-short branch July 25, 2023 15:06
evan-forbes added a commit that referenced this pull request Jul 26, 2023
…2163)

## Overview

the testnode is still hitting the race detector #1369, so we aren't
running those integration tests with the race detector

## Checklist

- [x] New and updated code has appropriate documentation
- [x] New and updated code has new and/or updated testing
- [x] Required CI checks are passing
- [x] Visual proof for any user facing features like CLI or
documentation updates
- [ ] Linked issues closed with keywords
evan-forbes added a commit that referenced this pull request Aug 2, 2023
…2163)

## Overview

the testnode is still hitting the race detector #1369, so we aren't
running those integration tests with the race detector

## Checklist

- [x] New and updated code has appropriate documentation
- [x] New and updated code has new and/or updated testing
- [x] Required CI checks are passing
- [x] Visual proof for any user facing features like CLI or
documentation updates
- [ ] Linked issues closed with keywords
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI item that directly relates to or modify the CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants