Skip to content

Conversation

@subtleGradient
Copy link

This commit adds the AGENTS.md file, which contains guidelines for contributing to the project. This includes information on project structure, build and test commands, coding style, testing guidelines, and commit/pull request guidelines.

This commit adds the AGENTS.md file, which contains guidelines for contributing to the project. This includes information on project
structure, build and test commands, coding style, testing guidelines, and commit/pull request guidelines.
Copilot AI review requested due to automatic review settings September 18, 2025 23:03
Copy link

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 adds comprehensive contribution guidelines to the project by introducing an AGENTS.md file that documents project structure, build commands, coding style, testing procedures, and commit conventions for the libansilove C library.

Key changes:

  • Added AGENTS.md with detailed contributor guidelines covering project structure, build/test commands, coding standards, and PR requirements
  • Added a CLAUDE.md file that appears to reference AGENTS.md

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
AGENTS.md Comprehensive contribution guidelines covering project structure, build commands, coding style, testing, and commit conventions
CLAUDE.md Single line file containing "AGENTS.md" reference

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@@ -0,0 +1 @@
AGENTS.md No newline at end of file
Copy link

Copilot AI Sep 18, 2025

Choose a reason for hiding this comment

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

This file contains only a filename reference without any context or explanation. Consider adding documentation explaining the purpose of this file or removing it if it's not needed.

Copilot uses AI. Check for mistakes.
Comment on lines +4 to +5
libansilove is a C library that converts ANSI and related art files to PNG. Core headers live in `include/`, while the implementation sits in `src/` with `loaders/` containing format-specific decoders and `fonts/` bundling built-in typefaces. Cross-platform fallbacks are under `compat/`. The `example/` directory shows how to invoke the API end-to-end, and `man/` provides installed manual pages. Dedicated fuzzing harnesses reside in `fuzz/`; build them only when running sanitizer-heavy tests.

Copy link

Copilot AI Sep 18, 2025

Choose a reason for hiding this comment

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

[nitpick] This paragraph is extremely long and dense, making it difficult to read and understand. Consider breaking it into multiple sentences or using bullet points to improve readability.

Suggested change
libansilove is a C library that converts ANSI and related art files to PNG. Core headers live in `include/`, while the implementation sits in `src/` with `loaders/` containing format-specific decoders and `fonts/` bundling built-in typefaces. Cross-platform fallbacks are under `compat/`. The `example/` directory shows how to invoke the API end-to-end, and `man/` provides installed manual pages. Dedicated fuzzing harnesses reside in `fuzz/`; build them only when running sanitizer-heavy tests.
libansilove is a C library that converts ANSI and related art files to PNG. The project is organized as follows:
- Core headers live in `include/`.
- The main implementation is in `src/`.
- `loaders/` contains format-specific decoders.
- `fonts/` bundles built-in typefaces.
- Cross-platform fallbacks are under `compat/`.
- The `example/` directory shows how to invoke the API end-to-end.
- `man/` provides installed manual pages.
- Dedicated fuzzing harnesses reside in `fuzz/` (build them only when running sanitizer-heavy tests).

Copilot uses AI. Check for mistakes.
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.

1 participant