Skip to content
Open

Main #14

Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 27 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
# Changelog

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

## Unreleased

### Added
- Exported `io` module (atomic) to support direct IO integration and library consumers.

Copilot AI Nov 2, 2025

Copy link

Choose a reason for hiding this comment

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

Corrected spelling of 'IO' to 'I/O'. When referring to input/output operations, the standard abbreviation uses a forward slash and is typically written in uppercase as 'I/O'.

Suggested change
- Exported `io` module (atomic) to support direct IO integration and library consumers.
- Exported `io` module (atomic) to support direct I/O integration and library consumers.

Copilot uses AI. Check for mistakes.
- Snapshot updates for glyph formatting examples.

### Changed
- Removed library-related references and cleanup from internal modules (work done on branch `fix/remove-librs`).
- Improved formatting stability for example outputs (snapshots updated).

### Fixed
- Various fixes that surfaced during removal/cleanup of library references.

### Tests
- All unit and integration tests passing locally:
- 34 unit tests for core library passed.
- Multiple integration/golden tests passed.
- Snapshot review performed and updated for `glyph.ai.snap`.
- Note: Some non-critical compiler warnings remain (unused variables and a dead method). These do not affect correctness but should be cleaned up in a follow-up.

### Notes
- Recommend a minor version bump (e.g. v0.2.0 -> v0.2.1) after merge if you consider these changes a patch/minor release.

Copilot AI Nov 2, 2025

Copy link

Choose a reason for hiding this comment

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

The version bump recommendation is inconsistent. A version change from v0.2.0 to v0.2.1 is a patch release (third number increments), not a minor release. The text should clarify whether this is intended as a patch or minor release. If minor, the example should show v0.2.0 -> v0.3.0.

Suggested change
- Recommend a minor version bump (e.g. v0.2.0 -> v0.2.1) after merge if you consider these changes a patch/minor release.
- Recommend a patch version bump (e.g. v0.2.0 -> v0.2.1) if you consider these changes a patch release, or a minor version bump (e.g. v0.2.0 -> v0.3.0) if you consider them a minor release.

Copilot uses AI. Check for mistakes.
- CI should be allowed to run on the PR to verify platform-specific behavior.