Skip to content

Commit 3397ac5

Browse files
Copilotkraenhansen
andcommitted
Add focused Rust/Ferric integration patterns to copilot instructions
Co-authored-by: kraenhansen <1243959+kraenhansen@users.noreply.github.com>
1 parent 3966991 commit 3397ac5

File tree

2 files changed

+12
-177
lines changed

2 files changed

+12
-177
lines changed

.github/copilot-instructions.md

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,4 +74,16 @@ Library names use double-dash separation: `package-name--path-component--addon-n
7474
- **Cross-package**: Use root-level `npm test` for cross-package testing once individual package tests pass
7575
- **Mobile integration**: Available but not the primary AI development focus - ask the developer to run those tests as needed
7676

77+
## Rust/Ferric Integration Patterns
78+
79+
### Target Management
80+
- Use `Set<TargetName>` for target collections, leverage `Set.intersection()` and `Set.has()`
81+
- Rename functions when responsibilities change: `ensureInstalledTargets``ensureAvailableTargets`
82+
- Separate tier 3 target validation into dedicated functions like `assertNightlyToolchain()`
83+
84+
### Workflow Development
85+
- Add jobs to existing `.github/workflows/check.yml` instead of creating new workflow files
86+
- Use conditional execution: `if: github.ref == 'refs/heads/main' || contains(github.event.pull_request.labels.*.name, 'Ferric 🦀')`
87+
- Keep workflow steps minimal: install targets, install nightly, build and test
88+
7789
**Documentation**: Integration details, platform setup, and toolchain configuration are covered in existing repo documentation files.

COPILOT_INSTRUCTIONS_IMPROVEMENTS.md

Lines changed: 0 additions & 177 deletions
This file was deleted.

0 commit comments

Comments
 (0)