google-deepmind / google-deepmind/distrax

Add checksum verification for `wget .pylintrc`

Open
#298 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
651
Forks
48
Avg merge
21h 4m
Merged PRs (30d)
3

Description

@hbq1 Should we add quick checksum verification in `test.sh` when downloading a file to enforce integrity verification?

https://github.com/google-deepmind/distrax/blob/1f7ef15af2a5cccb05c4ccf561f97a6e2b1c4bd7/test.sh#L39

Currently it says:
```
# Download Google OSS config.
wget -nd -v -t 3 -O .pylintrc https://google.github.io/styleguide/pylintrc
```

cc @mihaimaruseac

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.