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

chore: remove unused dependency features from fedimint-core #4868

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tvolk131
Copy link
Contributor

@tvolk131 tvolk131 commented Apr 7, 2024

No description provided.

@tvolk131 tvolk131 force-pushed the fedimint-core_cleanup_cargo_toml branch from 8a89914 to 1c02421 Compare April 18, 2024 16:31
@tvolk131 tvolk131 marked this pull request as ready for review April 18, 2024 18:28
@tvolk131 tvolk131 requested a review from a team as a code owner April 18, 2024 18:28
dpc
dpc previously approved these changes Apr 18, 2024
@dpc dpc added this pull request to the merge queue Apr 18, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 18, 2024
@elsirion elsirion added this pull request to the merge queue Apr 19, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Apr 19, 2024
@dpc
Copy link
Contributor

dpc commented Apr 19, 2024

Merge conflict, oof.

@tvolk131
Copy link
Contributor Author

Oooh, so close!

Rebased

dpc
dpc previously approved these changes Apr 19, 2024
@dpc dpc enabled auto-merge April 19, 2024 17:09
@dpc
Copy link
Contributor

dpc commented Apr 19, 2024

wasm broken

auto-merge was automatically disabled April 22, 2024 15:25

Head branch was pushed to by a user without write access

@tvolk131 tvolk131 force-pushed the fedimint-core_cleanup_cargo_toml branch from 4018821 to 1e6d6e7 Compare April 22, 2024 15:25
dpc
dpc previously approved these changes Apr 22, 2024
@dpc dpc enabled auto-merge April 22, 2024 19:12
@dpc
Copy link
Contributor

dpc commented Apr 22, 2024

@tvolk131 Ping me directly on Discord if you ever need it quickly approved, so we can get it through somehow. :D

@elsirion
Copy link
Contributor

@tvolk131 Ping me directly on Discord if you ever need it quickly approved, so we can get it through somehow. :D

Same. (currently WASM is still broken though)

auto-merge was automatically disabled April 26, 2024 15:31

Head branch was pushed to by a user without write access

@tvolk131 tvolk131 force-pushed the fedimint-core_cleanup_cargo_toml branch from cfaa93c to ddc9936 Compare April 26, 2024 15:31
dpc
dpc previously approved these changes Apr 26, 2024
@elsirion
Copy link
Contributor

Cross-compiling is failing.

@dpc
Copy link
Contributor

dpc commented Apr 29, 2024

serde feature on url somewhere missing

dpc
dpc previously approved these changes May 7, 2024
@tvolk131 tvolk131 marked this pull request as draft May 7, 2024 16:06
dpc
dpc previously approved these changes May 21, 2024
@elsirion
Copy link
Contributor

I pushed a suspected fix for the CI failure, feel free to squash/delete if it works/doesn't.

This was probably pulled in by another dependency previously.
@elsirion elsirion force-pushed the fedimint-core_cleanup_cargo_toml branch from ea45f0a to c45b799 Compare May 23, 2024 11:09
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.

None yet

3 participants