Skip to content

feat!: create new crate layout#923

Merged
indietyp merged 14 commits intomasterfrom
bm/init-crate-layout
Dec 5, 2025
Merged

feat!: create new crate layout#923
indietyp merged 14 commits intomasterfrom
bm/init-crate-layout

Conversation

@indietyp
Copy link
Member

@indietyp indietyp commented Dec 2, 2025

Creates the necessary crate layout.

Additionally does:

  • Upgrade to Edition 2024
  • Set up mise for external tools
  • Format toml files with taplo
  • Update resolver to version 3
  • Bump MSRV to 1.91
  • Specify toolchain (currently 2025-12-02)
  • Specify rustfmt configuration

BREAKING CHANGE: Initialize pegraph-core

@indietyp indietyp requested a review from RaoulLuque December 2, 2025 21:19
@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2025

This PR contains breaking changes to the public Rust API.

cargo-semver-checks summary

Copy link
Collaborator

Choose a reason for hiding this comment

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

Hi, glad to see this PR! I only suggest renaming
GSI --> Gsi
GSISMall --> GsiSmall
in serialization-tests/tests/serialization.rs / from_json_digraph_str_i32 function.

to satisfy just ci runs.
(Or add a skip of the appropriate lint)

Copy link
Member

@RaoulLuque RaoulLuque left a comment

Choose a reason for hiding this comment

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

Thanks for creating the PR ! 🦕
Looks great overall, just a few questions / suggestions ^^

@RaoulLuque RaoulLuque force-pushed the bm/init-crate-layout branch from 802bd7d to e775168 Compare December 3, 2025 16:58
@RaoulLuque
Copy link
Member

Accidentally pushed to your branch, sry ^^

@RaoulLuque
Copy link
Member

RaoulLuque commented Dec 4, 2025

Note that I cancelled the Unsoundness check fast, as it seems to be running for ~4 hours for some reason. #924 fixes this (by temporarily disabling the job). So be careful when pushing new changes without the fix 👍

@indietyp indietyp enabled auto-merge December 5, 2025 17:59
@indietyp indietyp added this pull request to the merge queue Dec 5, 2025
Merged via the queue into master with commit bbfc381 Dec 5, 2025
9 checks passed
@indietyp indietyp deleted the bm/init-crate-layout branch December 5, 2025 18:04
@github-actions github-actions bot mentioned this pull request Dec 5, 2025
This was referenced Dec 9, 2025
@github-actions github-actions bot mentioned this pull request Feb 7, 2026
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.

3 participants