search

qurator-spk / dinglehopper

An OCR evaluation tool
Apache License 2.0
58 stars 12 forks source link

No tests are run for PRs? #68

Closed mikegerber closed 10 months ago

mikegerber commented 2 years ago

67 caused the code to fail, and the tests weren't run for the PR. Why?

kba commented 2 years ago

67 caused the code to fail, and the tests weren't run for the PR. Why?

Pull requests from branches in other remotes are not tested in Circle CI by default for security reasons (they could extract secrets like passwords and tokens). You have to enable "Build forked pull requests" in the settings.

mikegerber commented 2 years ago

Pull requests from branches in other remotes are not tested in Circle CI by default for security reasons (they could extract secrets like passwords and tokens). You have to enable "Build forked pull requests" in the settings.

Thanks! I'll need to think about this.

mikegerber commented 10 months ago

Fixed in #96.

Needs more review for security: I checked the security guides and updated some settings but with this stuff it's best to look twice.