Skip to content

chore: remove mb-rs as dependancy#20

Open
RustyNova016 wants to merge 4 commits into
metabrainz:masterfrom
RustyNova016:integrate_mb_rs
Open

chore: remove mb-rs as dependancy#20
RustyNova016 wants to merge 4 commits into
metabrainz:masterfrom
RustyNova016:integrate_mb_rs

Conversation

@RustyNova016

@RustyNova016 RustyNova016 commented Mar 21, 2025

Copy link
Copy Markdown
Collaborator
  • Remove mb-rs as dependancy as it is pulled for only 2 structs
  • Move the database structs into a new "models" folder. It's clearer that it's for all the database models. The structs are also sorted by origin (MB or Melba)

Require #19 to pass first

@RustyNova016 RustyNova016 mentioned this pull request Mar 21, 2025
2 tasks
@RustyNova016

Copy link
Copy Markdown
Collaborator Author

No idea why the test fail as I haven't changed anything remotely tokio related? Maybe a flaky test?

Probably should use nextest to check for those as well

Comment thread src/models/musicbrainz_db.rs Outdated
So funnily enough, the current crate had the tokio runtime feature enabled through mb_rs, and was the only thing preventing it from crashing and burning when spawning a `tokio::Task`

Renabled it, and it allows purging the async_std runtime which was pulled in for nothing
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