Skip to content

Conversation

@baileympearson
Copy link
Contributor

@baileympearson baileympearson commented Nov 12, 2024

Description

What is changing?

This PR removes all rust code in the repo.

Is there new documentation needed for these changes?

No.

What is the motivation for this change?

To have a clean slate on which to start over for the C++ rewrite.

Double check the following

  • Ran npm run format:js && npm run format:rs script
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: type(NODE-xxxx)[!]: description
    • Example: feat(NODE-1234)!: rewriting everything in coffeescript
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@baileympearson baileympearson marked this pull request as ready for review November 12, 2024 23:15
@nbbeeken nbbeeken self-assigned this Nov 13, 2024
@nbbeeken nbbeeken self-requested a review November 13, 2024 17:04
@nbbeeken nbbeeken changed the title feat!(NODE-6538): remove all Rust from mongodb-js/zstd feat!(NODE-6542): remove all Rust from mongodb-js/zstd Nov 13, 2024
@nbbeeken nbbeeken added the Primary Review In Review with primary reviewer, not yet ready for team's eyes label Nov 13, 2024
@nbbeeken nbbeeken merged commit e5f4fe5 into main Nov 13, 2024
@nbbeeken nbbeeken deleted the NODE-6538 branch November 13, 2024 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Primary Review In Review with primary reviewer, not yet ready for team's eyes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants