Skip to content

Conversation

@tisonkun
Copy link
Contributor

@tisonkun tisonkun commented Jan 3, 2026

No description provided.

Signed-off-by: tison <wander4096@gmail.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR reorganizes the examples from exn/examples to a new examples workspace member with sources in examples/src. This restructuring consolidates dependency management through workspace-level configurations and updates the build tooling accordingly.

Key Changes:

  • Moved examples from exn/examples/ to examples/src/ as a new workspace member
  • Centralized dependency versions in workspace configuration
  • Updated xtask to reference the new examples directory structure and explicitly use nightly toolchain for formatting/linting

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
Cargo.toml Added examples to workspace members and introduced workspace-level dependency management
examples/Cargo.toml New manifest defining the examples package with proper example targets
examples/src/*.rs Updated file path references in comments from exn/examples/ to examples/src/
exn/Cargo.toml Migrated to workspace dependencies and removed derive_more (now in examples)
xtask/Cargo.toml Migrated to workspace dependencies
xtask/src/main.rs Updated examples directory path and added explicit +nightly to cargo fmt/clippy commands
.github/workflows/ci.yml Updated to actions/checkout@v6 and added nightly to test matrix
Cargo.lock Reflects dependency updates including new transitive dependencies from insta upgrade

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tisonkun tisonkun merged commit bb4d8ea into main Jan 4, 2026
15 checks passed
@tisonkun tisonkun deleted the examples branch January 4, 2026 02:16
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.

2 participants