Skip to content

Update directory and ffi toml files where deps were incorrect - #1044

Merged
arminsabouri merged 1 commit into
payjoin:masterfrom
benalleng:cli-directory-dep-drift
Sep 5, 2025
Merged

arminsabouri merged 1 commit into
payjoin:masterfrom
benalleng:cli-directory-dep-drift

Conversation

@benalleng

@benalleng benalleng commented Sep 4, 2025 •

Copy link
Copy Markdown
Collaborator

There were 2 deps in the directory and ffi where the versions listed were off compared to the rest of the repo.

The offenders
payjoin-directory tokio: 1.23.31 (doesn't exist) and
uniffi: 0.29.1 (the build dep differed from the main)

Pull Request Checklist

Please confirm the following before requesting review:

@benalleng benalleng changed the title Update directory and cli toml files where deps were incorrect Update directory and ffi toml files where deps were incorrect Sep 4, 2025
There were 2 deps in the directory and ffi where the versions listed
were off compared to the rest of the repo.

The offenders payjoin-directory tokio: 1.23.31 (doesn't exist) and
uniffi: 0.29.1 (the build dep differed from the main)
@benalleng
benalleng force-pushed the cli-directory-dep-drift branch from 0c88670 to be86689 Compare September 4, 2025 19:12
@coveralls

coveralls commented Sep 4, 2025 •

Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 17474248282

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 85.94%

Totals Coverage Status
Change from base Build 17446213698: 0.0%
Covered Lines: 8197
Relevant Lines: 9538

💛 - Coveralls

@arminsabouri arminsabouri left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK be86689

@benalleng For common deps (like tokio) would it make sense to just have workspace dependencies? Maybe not if different crates have different MSRV

@arminsabouri
arminsabouri merged commit 0616155 into payjoin:master Sep 5, 2025
14 checks passed
@benalleng

Copy link
Copy Markdown
Collaborator Author

Now that we have everything updated to 1.85.0 I think its probably ok to do this. Though I probably want to wait until we get the ohttp-relay merged into the mono repo

@benalleng
benalleng deleted the cli-directory-dep-drift branch October 6, 2025 18:40
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.

3 participants