8000 Make process-backend and process-reader private crates by marti4d · Pull Request #224 · rust-minidump/minidump-writer · GitHub
[go: up one dir, main page]

Skip to content

Make process-backend and process-reader private crates - #224

Closed
marti4d wants to merge 1 commit into
rust-minidump:mainfrom
marti4d:private-process-backend-and-reader
Closed

marti4d wants to merge 1 commit into
rust-minidump:mainfrom
marti4d:private-process-backend-and-reader

Conversation

@marti4d
@marti4d marti4d commented Aug 25, 2026
Copy link
Copy Markdown
Collaborator

These two crates were accidentally released on crates.io because we forgot to add the "publish = false" option to Cargo.toml. They are internal crates to minidump-writer and aren't really meant for public consumption, so it's unfortunate to have made that mistake.

This prevents us from accidentally doing this in the future.

(Note that, eventually, we may actually release process-reader as an independently-useful crate for the community... But it needs to be buffed up a bit before it's ready for that).

These two crates were accidentally released on crates.io because we forgot to
add the "publish = false" option to Cargo.toml. They are internal crates to
minidump-writer and aren't really meant for public consumption, so it's
unfortunate to have made that mistake.

This prevents us from accidentally doing this in the future.

(Note that, eventually, we may actually release `process-reader` as an
independently-useful crate for the community... But it needs to be buffed up
a bit before it's ready for that).
@marti4d
marti4d commented Aug 26, 2026
Copy link
Copy Markdown
Collaborator Author

Nevermind -- I see that we can't even publish the main crate on crates.io without these.

I think I'm going to have to rename them to minidump-writer-process-backend and minidump-writer-process-reader to avoid clogging up crates.io with private implementation details of our crate.

@marti4d marti4d closed this Aug 26, 2026
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

0