Skip to content

refactor: yet another rewrite#30

Closed
philiplinden wants to merge 28 commits into
mainfrom
yet-another-rewrite
Closed

refactor: yet another rewrite#30
philiplinden wants to merge 28 commits into
mainfrom
yet-another-rewrite

Conversation

@philiplinden

@philiplinden philiplinden commented Jun 17, 2025

Copy link
Copy Markdown
Owner

🔧 Chores

  • Update changelog

🐛 Bug Fixes

  • Clean up after the ai

📝 Documentation

  • More placeholders

♻️ Refactoring

  • Split into multiple crates

👷 CI/CD

  • Don't build for web
  • Try a different changelog approach
  • Maybe this time it will work
  • Tweak the git-cliff
  • Hopefully this works now
  • Decouple lints from tests, only dynamic linking in dev profile
  • Upgrade build and release workflows
  • Fix cl and ci issues
  • Fix cl formatting
  • Fix cl formatting
  • Update changelog triggers and build opts

📦 Build

  • Make some tweaks so it is easier to work with bevy cli

📦 Other Changes

  • Merge 05c224d into 0268037
  • Update .github/workflows/ci.yaml
  • Remove fancy logging
  • Move configs from buoy_sim to buoy_common
  • Clippy fix
  • Remove broken examples
  • Broken but closer to the real deal
  • Buoy_app is redundant to buoy_sim
  • Sketch out the repo structure with ai placeholders
  • Total rewrite, this time with docs
  • Actually run headless
  • Start devving a server-client arch, use best practices for bevy 16

@philiplinden
philiplinden requested a review from Copilot June 17, 2025 06:47
@philiplinden philiplinden self-assigned this Jun 17, 2025

Copilot AI left a comment

Copy link
Copy Markdown

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 refactors the repository’s CI/CD and configuration by replacing the monolithic build workflow with several specialized workflows (CI, changelog, docs, and release) and reorganizing the workspace structure. It also removes the previous build configuration and legacy crate source files, and adds an .editorconfig file to enforce consistent formatting.

  • Splits the build process into modular workflows.
  • Removes outdated core source files and Cargo.toml files for buoy-core and buoy-ui.
  • Updates the workspace configuration in the root Cargo.toml.

Reviewed Changes

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

Show a summary per file
File Description
crates/buoy-ui/Cargo.toml Removed; reflects a restructuring of UI-related configuration.
crates/buoy-core/* Multiple source files and Cargo.toml removed; indicates a reorganization of core modules.
.github/workflows/*.yaml Added new workflows (CI, changelog, docs, release) and removed the old build.yml workflow.
Cargo.toml Updated workspace members, edition, and dependency versions to support the new structure.
.editorconfig Added to enforce consistent formatting across the project.
clippy.toml, cliff.toml New configuration files to enforce linting and changelog standards.
README.md Updated project overview and documentation reflecting the new architecture and workflows.
CONTRIBUTING.md New document outlining code organization, style, testing, and contribution guidelines.

Comment thread .github/workflows/ci.yaml Outdated
philiplinden and others added 2 commits June 17, 2025 02:57
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@philiplinden
philiplinden force-pushed the yet-another-rewrite branch from 035ba32 to 0a4f9be Compare June 18, 2025 01:45
@github-actions github-actions Bot changed the title Yet another rewrite chore: update changelog Jun 18, 2025
@github-actions
github-actions Bot force-pushed the yet-another-rewrite branch from 0a4f9be to b8a16ca Compare June 18, 2025 01:45
@philiplinden
philiplinden force-pushed the yet-another-rewrite branch from b8a16ca to 1358d25 Compare June 18, 2025 01:53
@philiplinden philiplinden changed the title chore: update changelog refactor: yet another rewrite Jun 18, 2025
@philiplinden
philiplinden force-pushed the yet-another-rewrite branch from a903577 to d3cf822 Compare June 18, 2025 02:17
@philiplinden
philiplinden force-pushed the yet-another-rewrite branch from d3cf822 to 7340cf0 Compare June 18, 2025 02:39
@github-actions

Copy link
Copy Markdown

Changelog

All notable changes to this project will be documented in this file.

The format is based on Keep a Changelog,
and this project adheres to Semantic Versioning.

Unreleased

Bug Fixes

  • Clean up after the ai

Build

  • Make some tweaks so it is easier to work with bevy cli

CI

  • Fix cl and ci issues
  • Fix cl formatting
  • Fix cl formatting
  • Update changelog triggers and build opts

Chore

  • Update changelog

DEV

  • Total rewrite, this time with docs

Documentation

  • More placeholders

REPO

  • Sketch out the repo structure with ai placeholders

Refactor

  • Split into multiple crates

Buoy_common

  • Move configs from buoy_sim to buoy_common

Buoy_sim

  • Remove fancy logging
  • Clippy fix
  • Remove broken examples

Crates

  • Buoy_app is redundant to buoy_sim

Dev

  • Actually run headless
  • Start devving a server-client arch, use best practices for bevy 16

Repo

  • Broken but closer to the real deal

@philiplinden
philiplinden force-pushed the yet-another-rewrite branch 2 times, most recently from 70d485b to 0149e44 Compare June 18, 2025 03:53
@philiplinden
philiplinden force-pushed the yet-another-rewrite branch from 0149e44 to 6569c31 Compare June 18, 2025 03:59
@github-actions

github-actions Bot commented Jun 18, 2025

Copy link
Copy Markdown

📝 PR Changes

Changelog

All notable changes to this project will be documented in this file.

The format is based on Keep a Changelog,
and this project adheres to Semantic Versioning.

Unreleased

🔧 Chores

  • Update changelog

🐛 Bug Fixes

  • Winit is required for an app window to appear
  • Clean up after the ai

📝 Documentation

  • More placeholders

♻️ Refactoring

  • Split into multiple crates

👷 CI/CD

  • Don't build for web
  • Try a different changelog approach
  • Maybe this time it will work
  • Tweak the git-cliff
  • Hopefully this works now
  • Decouple lints from tests, only dynamic linking in dev profile
  • Upgrade build and release workflows
  • Fix cl and ci issues
  • Fix cl formatting
  • Fix cl formatting
  • Update changelog triggers and build opts

📦 Build

  • Make some tweaks so it is easier to work with bevy cli

📦 Other Changes

  • Merge abdf704 into 0268037
  • Update .github/workflows/ci.yaml
  • Remove fancy logging
  • Move configs from buoy_sim to buoy_common
  • Clippy fix
  • Remove broken examples
  • Broken but closer to the real deal
  • Buoy_app is redundant to buoy_sim
  • Sketch out the repo structure with ai placeholders
  • Total rewrite, this time with docs
  • Actually run headless
  • Start devving a server-client arch, use best practices for bevy 16

📦 Version Changes

No version changes detected in this PR.

📜 Full Changelog

git-cliff/CHANGELOG.md

@philiplinden
philiplinden force-pushed the yet-another-rewrite branch from 4719ded to 1926b4a Compare June 18, 2025 08:32
@philiplinden
philiplinden force-pushed the yet-another-rewrite branch 2 times, most recently from e1c9e86 to 3f8f513 Compare June 18, 2025 08:50
@philiplinden
philiplinden force-pushed the yet-another-rewrite branch 6 times, most recently from bbb54f8 to 55e7e79 Compare June 18, 2025 09:53
@philiplinden
philiplinden force-pushed the yet-another-rewrite branch from 55e7e79 to 05c224d Compare June 18, 2025 10:02
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