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.

Sign up for GitHub

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

Jump to bottom

remove unused files from marshalling #248

Merged
merged 1 commit into from
Jan 13, 2025
Merged

remove unused files from marshalling #248

merged 1 commit into from
Jan 13, 2025

Conversation

x-mass
Copy link
Member

@x-mass x-mass commented Jan 7, 2025

No description provided.

@x-mass x-mass force-pushed the refactor-marshalling branch from ba30cba to 16af62f Compare January 7, 2025 09:11
@x-mass x-mass requested a review from vo-nil January 7, 2025 09:17
@x-mass x-mass force-pushed the refactor-marshalling branch from 16af62f to ef68b58 Compare January 7, 2025 09:44
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 7, 2025 •
edited
Loading

Clang Test Results

  163 files  ±0    163 suites  ±0   14m 17s ⏱️ -4s
8 526 tests ±0  8 520 ✅ ±0  6 💤 ±0  0 ❌ ±0 
8 592 runs  ±0  8 586 ✅ ±0  6 💤 ±0  0 ❌ ±0 

Results for commit 732d3b0. ± Comparison against base commit 5eedb7c.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 7, 2025 •
edited
Loading

Gcc Test Results

  163 files  ±0    163 suites  ±0   15m 32s ⏱️ -8s
8 526 tests +4  8 520 ✅ +4  6 💤 ±0  0 ❌ ±0 
8 592 runs  +4  8 586 ✅ +4  6 💤 ±0  0 ❌ ±0 

Results for commit 732d3b0. ± Comparison against base commit 5eedb7c.

♻️ This comment has been updated with latest results.

@x-mass x-mass force-pushed the refactor-marshalling branch from 17b6822 to 701dc36 Compare January 7, 2025 12:29
@vo-nil vo-nil marked this pull request as ready for review January 10, 2025 12:36
@vo-nil vo-nil enabled auto-merge January 10, 2025 12:36
@vo-nil vo-nil added this pull request to the merge queue Jan 10, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Jan 10, 2025
@vo-nil vo-nil enabled auto-merge January 10, 2025 18:11
@vo-nil vo-nil force-pushed the refactor-marshalling branch from f7be2fc to 6fa0b63 Compare January 13, 2025 04:53
@vo-nil vo-nil requested a review from martun January 13, 2025 04:54
@x-mass
refactor: remove marshalling string_view
732d3b0 
refactor: remove marshalling field_cast

refactor: remove marshalling accumulator_set

docs: add how to avoid OOM to readme

refactor: remove marshalling array_view

fix: cmake for crypto3 checks
@x-mass x-mass force-pushed the refactor-marshalling branch from 6fa0b63 to 732d3b0 Compare January 13, 2025 06:10
@vo-nil vo-nil self-requested a review January 13, 2025 06:36
@vo-nil vo-nil added this pull request to the merge queue Jan 13, 2025
Merged via the queue into master with commit cdda3c1 Jan 13, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vo-nil vo-nil vo-nil approved these changes

@martun martun martun approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@x-mass @martun @vo-nil