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

fix: foreign investments bench #1572

Merged
merged 2 commits into from
Sep 27, 2023
Merged

fix: foreign investments bench #1572

merged 2 commits into from
Sep 27, 2023

Conversation

wischli
Copy link
Contributor

@wischli wischli commented Sep 27, 2023

Description

Fixes benchmarking CI after merging #1570 unwantedly before the entire CI finished (benches were still progressing, suppose those are optional which I was not aware of). Sorry about the inconvenience!

Checklist:

  • I have added Rust doc comments to structs, enums, traits and functions
  • I have made corresponding changes to the documentation
  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that my feature works

@wischli wischli added D0-ready Pull request can be merged without special precaution and notification. I12-ci Issue related to CI labels Sep 27, 2023
@wischli wischli self-assigned this Sep 27, 2023
@wischli wischli enabled auto-merge (squash) September 27, 2023 14:46
@wischli wischli merged commit 4aae695 into main Sep 27, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D0-ready Pull request can be merged without special precaution and notification. I12-ci Issue related to CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants