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.

Sign up for GitHub

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

Jump to bottom

Port 1.8.1 changelog to develop #5902

Merged
merged 5 commits into from
Apr 16, 2021
Merged

Port 1.8.1 changelog to develop #5902

merged 5 commits into from
Apr 16, 2021

Conversation

emkll
Copy link
Contributor

@emkll emkll commented Apr 15, 2021

Status

Ready for review

Description of Changes

Closes #5891

Testing

@emkll emkll requested a review from kushaldas April 15, 2021 14:02
@emkll emkll requested a review from a team as a code owner April 15, 2021 14:02
@lgtm-com
Copy link

lgtm-com bot commented Apr 15, 2021

This pull request fixes 5 alerts when merging eef33a8 into 20dfdab - view on LGTM.com

fixed alerts:

  • 5 for Variable defined multiple times

@emkll emkll force-pushed the port-181-changes branch from eef33a8 to ac7b5df Compare April 15, 2021 14:38
@lgtm-com
Copy link

lgtm-com bot commented Apr 15, 2021

This pull request fixes 5 alerts when merging ac7b5df into 20dfdab - view on LGTM.com

fixed alerts:

  • 5 for Variable defined multiple times

zenmonkeykstop and others added 4 commits April 15, 2021 11:21
@zenmonkeykstop @emkll
Updated restore playbook to validate tor config after v2 removal
dfa511c 
(cherry picked from commit 2c37173)
@zenmonkeykstop @emkll
removed stray HiddenServiceDir directive
b71cdd8 
(cherry picked from commit c8b432f)
@emkll
Update Focal and Xenial builders
08b5e19 
Builder images needed updates

(cherry picked from commit 244d850)
@emkll
SecureDrop 1.8.1
196faa8 
(cherry picked from commit 71cbfa9)
@emkll emkll force-pushed the port-181-changes branch from ac7b5df to 196faa8 Compare April 15, 2021 15:28
@lgtm-com
Copy link

lgtm-com bot commented Apr 15, 2021

This pull request fixes 5 alerts when merging 196faa8 into 20dfdab - view on LGTM.com

fixed alerts:

  • 5 for Variable defined multiple times

kushaldas
kushaldas approved these changes Apr 16, 2021
View reviewed changes
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.

The PR is good, changelog contains all the update.

@kushaldas kushaldas merged commit fedce7a into develop Apr 16, 2021
@kushaldas kushaldas deleted the port-181-changes branch April 16, 2021 05:41
@eloquence eloquence mentioned this pull request Apr 23, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kushaldas kushaldas kushaldas approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Release SecureDrop 1.8.1
3 participants
@emkll @kushaldas @zenmonkeykstop