Skip to content

Add CLAUDE.md for Claude Code guidance#579

Merged
joshuahannan merged 4 commits intomasterfrom
josh/claude
Feb 17, 2026
Merged

Add CLAUDE.md for Claude Code guidance#579
joshuahannan merged 4 commits intomasterfrom
josh/claude

Conversation

@joshuahannan
Copy link
Member

@joshuahannan joshuahannan commented Feb 17, 2026

Adds claude config

joshuahannan and others added 2 commits February 17, 2026 11:12
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
CLAUDE.md Outdated
Copy link
Member

Choose a reason for hiding this comment

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

If you put this file contents in AGENTS.md and symlink CLAUDE.md->AGENTS.md, this will work for people using other tools (OpenAI Codex etc.). It seems like they're moving toward standardizing on AGENTS.md but still not compatible afaik.

CLAUDE.md Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
This repository contains the core smart contracts for the Flow blockchain, written in Cadence. The `master` branch contains the Cadence 1.0 version (not yet deployed to mainnet/testnet - see `cadence-0.42` branch for deployed versions).
This repository contains the core smart contracts for the Flow blockchain, written in Cadence.

outdated?

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah, from the readme. I'm updating this and the readme

CLAUDE.md Outdated
Comment on lines 18 to 19
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Run a single Cadence test file
flow test tests/dkg_test.cdc
# Run a single Cadence test file
flow test tests/dkg_test.cdc
# Run a specific Cadence test case
flow test tests/dkg_test.cdc --name "testResultSubmissionInit_InvalidGroupKeyLength"

CLAUDE.md Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Cadence tests deploy contracts using `Test.deployContract()` and account `0x0000000000000007` as the default admin.
Cadence tests deploy contracts using `Test.deployContract()` and account `0x0000000000000007` as the default admin.
New tests should be written in Cadence where possible.

I'm guessing that is true?

Copy link
Member Author

Choose a reason for hiding this comment

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

yes

@joshuahannan joshuahannan merged commit 22471b4 into master Feb 17, 2026
2 checks passed
@joshuahannan joshuahannan deleted the josh/claude branch February 17, 2026 18:24
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.

2 participants

Comments