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

Removes VirtualBox references throughout docs #216

Merged
merged 2 commits into from May 19, 2021

Conversation

conorsch
Copy link
Contributor

@conorsch conorsch commented May 4, 2021

Status

Ready for review

Description of Changes

Supplements freedomofpress/securedrop#5922. Edits the dev docs to remove explicit mentions of Virtualbox, preferring Libvirt (or Qubes) for any of the VM docs. Mostly this affects the macOS docs. For macOS, we're now more clearly directing folks to use the container dev env.

Testing

Release

No concerns, these changes are dev-only.

Checklist (Optional)

  • Doc linting (make docs-lint) passed locally
  • Doc link linting (make docs-linkcheck) passed
  • You have previewed (make docs) docs at http://localhost:8000

@conorsch
Copy link
Contributor Author

conorsch commented May 4, 2021

Marking as blocked for now, since I based these changes on #214

@eloquence
Copy link
Member

(Can be rebased now that #214 is merged.)

Supplements freedomofpress/securedrop#5922.
Edits the dev docs to remove explicit mentions of Virtualbox, preferring
Libvirt (or Qubes) for any of the VM docs. Mostly this affects the macOS
docs. For macOS, we're now more clearly directing folks to use the
container dev env.
@conorsch
Copy link
Contributor Author

Rebased, and removed "blocked" label, since #214 is in. Ready for review.

Signed-off-by: Allie Crevier <allie@freedom.press>
Copy link
Contributor

@sssoleileraaa sssoleileraaa left a comment

Choose a reason for hiding this comment

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

lgtm - conor also reviewed my small commit

@sssoleileraaa sssoleileraaa merged commit 75de6d3 into main May 19, 2021
SecureDrop Team Board automation moved this from Ready for Review to Done May 19, 2021
@sssoleileraaa sssoleileraaa deleted the remove-virtualbox-docs branch May 19, 2021 23:03
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