Skip to content
This repository was archived by the owner on Sep 21, 2024. It is now read-only.

Conversation

@jsantell
Copy link
Contributor

@jsantell jsantell commented Oct 19, 2023

Preparing with in-flight PR getditto/safer_ffi#191.

There's a new clippy lint rule (documented as available since 1.72.0, though only appears in nightly channels), and some extra finesse on targeting nightly builds with a dynamic lint. also fixed in latest safer_ffi

Other than that, just a lifetime change needed.

@jsantell jsantell force-pushed the nightly-build branch 4 times, most recently from f31e4a6 to 66c8888 Compare October 19, 2023 18:01
@jsantell jsantell requested a review from cdata October 19, 2023 18:04
@jsantell jsantell marked this pull request as ready for review October 25, 2023 16:22
@jsantell
Copy link
Contributor Author

Upstream getditto/safer_ffi#191 landed in [email protected], moving from draft PR

@jsantell jsantell force-pushed the nightly-build branch 4 times, most recently from ade2f21 to a021b4c Compare October 31, 2023 19:29
@jsantell
Copy link
Contributor Author

rebased on #685, updated with published safer-ffi 0.1.4 containing nightly fix, as well as the removal of a lint rule skip (also fixed in 0.1.4)

@github-actions
Copy link
Contributor

github-actions bot commented Oct 31, 2023

Test flake analysis

status platform features toolchain
🟢 macos-13 test-kubo,headers,rocksdb stable
🟢 windows-latest test-kubo,headers stable
🟢 ubuntu-latest test-kubo,headers,rocksdb stable
🟢 ubuntu-latest test-kubo,headers nightly
🟡 macos-13 test-kubo,headers stable
🟢 ubuntu-latest test-kubo,headers stable

Flake summary for macos-13, test-kubo,headers, stable

     Summary [  98.375s] 175 tests run: 175 passed (1 flaky), 1 skipped
   FLAKY 2/6 [  40.914s] noosphere::distributed_stress latency::clients_can_sync_when_there_is_a_lot_of_content

Copy link
Collaborator

@cdata cdata left a comment

Choose a reason for hiding this comment

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

LGTM

@cdata cdata merged commit 344dd7a into main Nov 6, 2023
@cdata cdata deleted the nightly-build branch November 6, 2023 18:02
@github-actions github-actions bot mentioned this pull request Nov 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants