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

Factor out fs-metadata and fs-properties crates from fs-storage #14

Merged
merged 4 commits into from
Mar 16, 2024

Conversation

tareknaser
Copy link
Collaborator

Fixes #12

tareknaser pushed a commit to tareknaser/ark-rust that referenced this pull request Mar 12, 2024
* Bump arklib version
* Added tokio to deal with futures
* Updating the index when links are added
* Update GitHub Actions
---------
Co-authored-by: Alvi Hysa <[email protected]>
tareknaser pushed a commit to tareknaser/ark-rust that referenced this pull request Mar 12, 2024
* Bump arklib version
* Added tokio to deal with futures
* Updating the index when links are added
* Update GitHub Actions
---------
Co-authored-by: Alvi Hysa <[email protected]>
@tareknaser
Copy link
Collaborator Author

tareknaser commented Mar 13, 2024

  • Fixed CI build issue. Seems to be related to dependency versioning.

tareknaser pushed a commit to tareknaser/ark-rust that referenced this pull request Mar 14, 2024
* Bump arklib version
* Added tokio to deal with futures
* Updating the index when links are added
* Update GitHub Actions
---------
Co-authored-by: Alvi Hysa <[email protected]>
The bench feature has been disabled for all crates within the workspace to resolve an issue encountered during CI testing.

The reason is that by default, Cargo implicitly adds a libtest benchmark harness to any crate when benchmarking, to handle any #[bench] functions, even if we have none. This ruins our Benchmarking CI with the error (Unrecognized option: save-baseline) in the logs.

For more details: https://bheisler.github.io/criterion.rs/book/faq.html#cargo-bench-gives-unrecognized-option-errors-for-valid-command-line-options

Signed-off-by: Tarek <[email protected]>
@tareknaser
Copy link
Collaborator Author

Fixed the issue with the benchmarks CI. The change needs to be in main as well for the CI test to work but can be validated here in my fork: tareknaser#3 (comment) . See commit message for more details

@tareknaser tareknaser requested a review from kirillt March 14, 2024 12:56
@kirillt kirillt merged commit 8e0a80b into main Mar 16, 2024
1 of 2 checks passed
@tareknaser tareknaser deleted the fs-metadata branch April 4, 2024 19:39
tareknaser pushed a commit to tareknaser/ark-rust that referenced this pull request Apr 6, 2024
* Bump arklib version
* Added tokio to deal with futures
* Updating the index when links are added
* Update GitHub Actions
---------
Co-authored-by: Alvi Hysa <[email protected]>
tareknaser pushed a commit to tareknaser/ark-rust that referenced this pull request Apr 6, 2024
* Bump arklib version
* Added tokio to deal with futures
* Updating the index when links are added
* Update GitHub Actions
---------
Co-authored-by: Alvi Hysa <[email protected]>
tareknaser pushed a commit that referenced this pull request Apr 8, 2024
* Bump arklib version
* Added tokio to deal with futures
* Updating the index when links are added
* Update GitHub Actions
---------
Co-authored-by: Alvi Hysa <[email protected]>
tareknaser pushed a commit to tareknaser/ark-rust that referenced this pull request Apr 9, 2024
* Bump arklib version
* Added tokio to deal with futures
* Updating the index when links are added
* Update GitHub Actions
---------
Co-authored-by: Alvi Hysa <[email protected]>
kirillt added a commit that referenced this pull request Apr 10, 2024
* Bump arklib version
* Added tokio to deal with futures
* Updating the index when links are added
* Update GitHub Actions
---------
Co-authored-by: Alvi Hysa <[email protected]>
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.

Create fs-metadata and fs-properties crates
2 participants