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

chore: uncomment out tests #1015

Merged
merged 1 commit into from
Nov 16, 2022
Merged

chore: uncomment out tests #1015

merged 1 commit into from
Nov 16, 2022

Conversation

evan-forbes
Copy link
Member

@evan-forbes evan-forbes commented Nov 16, 2022

Overview

apparently I forgot to un-comment some important tests

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 testing items that are strictly related to adding or extending test coverage label Nov 16, 2022
@evan-forbes evan-forbes self-assigned this Nov 16, 2022
@evan-forbes evan-forbes enabled auto-merge (squash) November 16, 2022 15:47
@evan-forbes evan-forbes merged commit 03bbff8 into main Nov 16, 2022
@evan-forbes evan-forbes deleted the evan/uncomment-out-tests branch November 16, 2022 15:56
rach-id pushed a commit to rach-id/celestia-app that referenced this pull request Nov 17, 2022
## Overview

apparently we forgot to un-comment some important tests

## 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
- [x] Linked issues closed with keywords
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing items that are strictly related to adding or extending test coverage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants