Skip to content

docs: update AGENTS.md and CONTRIBUTING.md#3

Merged
nknapp merged 6 commits into
mainfrom
cfgsync-agent
Jul 11, 2026
Merged

docs: update AGENTS.md and CONTRIBUTING.md#3
nknapp merged 6 commits into
mainfrom
cfgsync-agent

Conversation

@nknapp

@nknapp nknapp commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Various documentation improvements to AGENTS.md and CONTRIBUTING.md.

Comment thread e2e-tests/lib/env.ts Outdated
Comment thread e2e-tests/lib/env.ts Outdated
Comment thread e2e-tests/test-state-file-structure.test.ts Outdated
Comment thread e2e-tests/test-state-file-structure.test.ts Outdated
Comment thread e2e-tests/test-state-file-structure.test.ts Outdated

@nknapp nknapp left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

comments

Comment thread e2e-tests/test-state-file-structure.test.ts Outdated

@nknapp nknapp left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

te


const stateToml = await testbed.readTextFile("config.cfgsync.state");

const username = new TextDecoder().decode(

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

move this into the testbed

const {testbed, testDir, username, groupname, hash } = await Testbed.create()

Comment thread e2e-tests/test-state-file-structure.test.ts Outdated
'group = "TARGET_ABS"',
);
assertEquals(actual.trim(), expected.trim());
assertEquals(stateToml.trim(), expected.trim());

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

is the trim really necessary?

@nknapp
nknapp merged commit eb0d234 into main Jul 11, 2026
3 checks passed
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