Skip to content

Edits to AGENTS.md file#4868

Open
mwichmann wants to merge 1 commit into
SCons:masterfrom
mwichmann:infra/agents
Open

Edits to AGENTS.md file#4868
mwichmann wants to merge 1 commit into
SCons:masterfrom
mwichmann:infra/agents

Conversation

@mwichmann

Copy link
Copy Markdown
Collaborator

These are agent-suggested updates (OpenCode with Qwen 3.6), plus some edits of mine to the result.

In general terms, it suggested removing thing which were instructions to human readers, and some other boilerplate already covered in files it was going to read anyway (README).

I mainly filled out the instructions on doc building, because a question I asked left an agent now knowing how to validate a manpage change.

This has no effect on SCons itself - no docs, code or tests are modified.

@mwichmann mwichmann added the maintenance Tasks to maintain internal SCons code/tools label Jun 26, 2026
echo "test/ninja/ninja_handle_control_c_rebuild.py" >> .excludes
echo "test/ninja/shutdown_scons_daemon.py" >> .excludes
echo "test/scons-time/run/option/verbose.py" >> .excludes
elif [[ "${{matrix.os}}" == "windows"* ]]; then

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm guessing this was accidently left in this PR? Can you restore to original?

These are agent-suggested updates (OpenCode with Qwen 3.6),
plus some edits of mine to the result.

In general terms, it suggested removing thing which were instructions
to human readers, and some other boilerplate already covered in files
it was going to read anyway (README).

I mainly filled out the instructions on doc building, because a question
I asked left an agent now knowing how to validate a manpage change.

Signed-off-by: Mats Wichmann <mats@linux.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Tasks to maintain internal SCons code/tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants