Add build dep `wheel` to `requirements.txt`

NOAA-OWP / t-route

Tree based hydrologic and hydraulic routing

Other

43 stars 48 forks source link

Add build dep `wheel` to `requirements.txt` #782

Closed aaraney closed 3 months ago

aaraney commented 3 months ago

Should have added this in #780. This should fix the CI docker image build failures.

aaraney commented 3 months ago

Okay, it looks like when I manually installed wheel in the docker image before building t-route, that got rid of the build issues (see action).

Since we will pull in wheel from requirements.txt after this is merged, I am going to remove the "manual" install of wheel in the dockerfile. As a result, the action will fail again when run for the PR, but once we merge the issue should go away.

aaraney commented 3 months ago

Okay, just removed the temporary commit. We should be good to merge now.