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

feat: Fix nightly builds and add to CI. #690

Merged
merged 1 commit into from
Nov 6, 2023
Merged

feat: Fix nightly builds and add to CI. #690

merged 1 commit into from
Nov 6, 2023

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
rust/noosphere/build.rs Outdated Show resolved Hide resolved
@jsantell
Copy link
Contributor Author

Upstream getditto/safer_ffi#191 landed in safer_ffi@0.1.4, 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)

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
22 of 23 checks passed
@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 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

2 participants