Matt Fisher

See: https://github.com/nsidc/earthaccess/pull/587#discussion_r1626774207 cc @jhkennedy

We run Ruff in pre-commit.ci, but also it turns out in many other CI jobs :) I removed one extra run of Ruff in our CI last week or so:...

`earthaccess.download()` ignores errors

5

We found the `pqdm` library is the thing swallowing all the error messages. `pqdm` has an `exception_behavior` option, which defaults to `"ignore"`, surprisingly. The other two settings, `"deferred"` and `"immediate"`,...

>I ran into an error with Binder: > >``` >Solving environment: ...working... Error during build: ('Connection broken: IncompleteRead(0 bytes read)', IncompleteRead(0 bytes read)) >``` > >I believe Binder is on...

Experience reports are different than issue reports in that there isn't a presumption that something has gone wrong. I think often folks assume that mild annoyances or roadbumps are not...

Typecheck some API interfaces

2

I'm going to be changing a lot in the refactor to use Harmony. The typechecker is going to help find sneaky bugs along the way. This is a start, but...

Move dependencies from requirements files into pyproject.toml, add extra groups for dev, docs

5

Resolves #552 I hope you don't mind, I took a pass at the contributing doc and made some formatting fixes and re-organized things a little bit in a way that...

Remove redundant conditional

2

This conditional is handled by the next one in the chain.

Workflow to approve integration tests from forks

1

Follow up on #580. > [!IMPORTANT] > > [Preventing Pwn Requests is important](https://securitylab.github.com/resources/github-actions-preventing-pwn-requests/). > > Secrets exfiltration is a risk if we try to allow integration tests that use secrets...