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

cleanup: move ArchivalMaybeBinaryData to legacymodel #1336

Merged
merged 1 commit into from
Sep 28, 2023
Merged

Conversation

bassosimone
Copy link
Contributor

This diff concludes most of the development work associated with ooni/probe#2531. We move the nearly unused ArchivalMaybeBinaryData to the internal/legacy/legacymodel package to clearly mark it deprecated.

Finishing removing this struct is a task that is documented by ooni/probe#2543.

This diff concludes most of the development work associated with
ooni/probe#2531. We move the nearly unused
ArchivalMaybeBinaryData to the internal/legacy/legacymodel package
to clearly mark it deprecated.

Finishing removing this struct is a task that is documented by
ooni/probe#2543.
@bassosimone bassosimone merged commit a37f9c1 into master Sep 28, 2023
6 checks passed
@bassosimone bassosimone deleted the issue/2531 branch September 28, 2023 21:34
Murphy-OrangeMud pushed a commit to Murphy-OrangeMud/probe-cli that referenced this pull request Feb 13, 2024
This diff concludes most of the development work associated with
ooni/probe#2531. We move the nearly unused
ArchivalMaybeBinaryData to the internal/legacy/legacymodel package to
clearly mark it deprecated.

Finishing removing this struct is a task that is documented by
ooni/probe#2543.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant