Skip to content
This repository has been archived by the owner on Jan 5, 2024. It is now read-only.

use new reproducible wheel from packaging with build #85

Merged
merged 1 commit into from
Apr 7, 2021
Merged

Conversation

sssoleileraaa
Copy link
Contributor

Description

Now that freedomofpress/securedrop-builder#238 has been merged, update build-requirements.txt to reflect the new expected checksum for our reproducible idna wheel (build with build).

@conorsch
Copy link
Contributor

conorsch commented Apr 5, 2021

Pushed a temporary commit to ensure that once freedomofpress/securedrop-builder#239 is merged, CI will be happy here. We'll drop that commit before merging, but it's useful to observe the CI behavior.

@sssoleileraaa sssoleileraaa marked this pull request as ready for review April 5, 2021 23:31
kushaldas
kushaldas previously approved these changes Apr 6, 2021
Copy link
Contributor

@kushaldas kushaldas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved based on the tests mentioned in freedomofpress/securedrop-builder#239. No new wheels other than the PR itself.

@conorsch
Copy link
Contributor

conorsch commented Apr 7, 2021

CI is passing with temporary commit in https://circleci.com/gh/freedomofpress/securedrop-proxy/427. Will remove temporary commit, then merge.

@conorsch conorsch merged commit db9332f into main Apr 7, 2021
@conorsch conorsch deleted the update-reqs branch April 7, 2021 00:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants