Closed jianyangli closed 2 years ago
Hello @jianyangli! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
Looks like this PR should wait until after #338 is merged. Some test failures due to the use of @remote_data
, which will be fixed by #338
@mkelley I only changed 5 files instead of 9 as reported by the PR because rebase included those you changed in PR #338 . FYI for your review. Thanks.
Address #335