Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update requirements: securedrop-sdk 0.2.0 #1172

Merged
merged 3 commits into from Nov 3, 2020
Merged

update requirements: securedrop-sdk 0.2.0 #1172

merged 3 commits into from Nov 3, 2020

Conversation

sssoleileraaa
Copy link
Contributor

@sssoleileraaa sssoleileraaa commented Oct 27, 2020

Description

Bumps SDK to version 0.2.0. Just opening this PR even though we're not ready to use the latest SDK yet since I needed to update the client requirements in order to build the latest SDK wheels here: freedomofpress/securedrop-builder#204 (due to our package build logic)

@emkll
Copy link
Contributor

emkll commented Oct 28, 2020

After merging freedomofpress/securedrop-builder#204 and the new sdk wheels live on the pypi mirror https://pypi.securedrop.org/simple/securedrop-sdk/ which now unblocks work here.

We should update https://github.com/freedomofpress/securedrop-client/blob/update-to-sdk-0.2.0/requirements/build-requirements.txt#L12 prior to merging to ensure the new version of the wheel is included in the builds.

I just restarted CI after the pypi mirror was updated, there is what appears to be a valid test failure here.

@emkll emkll added this to In Development in SecureDrop Team Board Oct 28, 2020
@sssoleileraaa sssoleileraaa force-pushed the update-to-sdk-0.2.0 branch 3 times, most recently from 95c350d to be64d6e Compare October 28, 2020 16:25
@sssoleileraaa sssoleileraaa marked this pull request as ready for review October 28, 2020 16:25
@eloquence
Copy link
Member

I would suggest deferring merge until we tag the next client release, just out of conservatism (we don't need the changes in the SDK yet for the release bundled as SDW 0.5.0).

@sssoleileraaa sssoleileraaa force-pushed the update-to-sdk-0.2.0 branch 2 times, most recently from 324a25d to 1566588 Compare October 28, 2020 18:39
@sssoleileraaa
Copy link
Contributor Author

since we'll want functional tests that test (1) we are getting seen data from the server, and (2) we are sending seen data to the server, i'll want to add that to this pr instead: #1165

that way i can do functional tests all at once utilizing the new create_dev_data functionality that adds seen data automatically

@sssoleileraaa
Copy link
Contributor Author

This branch is out-of-date with the base branch

I change my mind. This repo is not busy enough to force this check.

@sssoleileraaa sssoleileraaa moved this from In Development to Ready for Review in SecureDrop Team Board Oct 29, 2020
@eloquence eloquence changed the title update requirments update requirements Oct 29, 2020
@sssoleileraaa
Copy link
Contributor Author

I change my mind. This repo is not busy enough to force this check.

The strict status check to enforce that branches are always up-to-date with the base branch before merge has been removed. Feels like a heavy weight has been lifted!

emkll
emkll previously approved these changes Nov 2, 2020
Copy link
Contributor

@emkll emkll left a comment

Choose a reason for hiding this comment

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

Thanks @creviera looks good to me. I'll update the mac requirements and submit another PR to resolve, but this is good to merge

@@ -9,7 +9,7 @@ pathlib2==2.3.2 --hash=sha256:460e67b14d0574b0529a0017b1eb05d10d9722681e303fec70
python-dateutil==2.7.5 --hash=sha256:f6eb9c17acd5a6954e1a5f2f999a41de3e7e25b6bc41baf6344bd053ec25ceeb
python-editor==1.0.3 --hash=sha256:e47dcec4ea883853b8196fbd425b875d7ec791d4ede2e20cfc70b9a25365c65b
requests==2.22.0 --hash=sha256:e3711bd465fd013abe428a8dade2938b578f05d2b06b0ae83daa98869c4548e8
securedrop-sdk==0.1.1 --hash=sha256:a631495acd741ab568410287879c5a3af3ccd38e00a2f3a127cc6b27cba99392
securedrop-sdk==0.2.0 --hash=sha256:dd0231e1b873197373a7e84e3610370634eead601223551153eca7cdd166e7d5
Copy link
Contributor

Choose a reason for hiding this comment

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

✔️

securedrop-sdk==0.1.1 \
--hash=sha256:138ce7a717db519c3c8d19b9475d7660fb7095d7608e8802723e682a7415e677 \
securedrop-sdk==0.2.0 \
--hash=sha256:c4a343077e8c0a38914e17f6369b830f1e361f9d66699b20803c07b39472357f \
Copy link
Contributor

Choose a reason for hiding this comment

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

✔️

Copy link
Contributor

@emkll emkll left a comment

Choose a reason for hiding this comment

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

Re-stamping after rebase fix to #1179

@emkll emkll merged commit 5cb61ec into main Nov 3, 2020
SecureDrop Team Board automation moved this from Ready for Review to Done Nov 3, 2020
@emkll emkll deleted the update-to-sdk-0.2.0 branch November 3, 2020 00:01
@emkll emkll changed the title update requirements update requirements: securedrop-sdk 0.2.0 Nov 3, 2020
@emkll emkll mentioned this pull request Nov 3, 2020
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants