Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 28, 2026

Extended .github/copilot-instructions.md with critical context missing from the original 171-line file. Added 154 lines covering gaps that would cause Copilot to make incorrect assumptions about codebase structure and conventions.

Additions

  • Code Style & Quality: Ruff configuration, legacy code handling, minimal-change philosophy
  • Architecture: Dual-interface design (wxPython GUI vs scriptable API), Fortran extension compilation model
  • CI/CD Context: Pixi-based workflows, test categorization (offline/network), platform matrix
  • Contributing Patterns: Pre-change checklist, when rebuilds are required, PATH management for executables
  • Common Patterns: Space group operations, lattice parameter handling, scriptable interface idioms
  • Gotchas: Build timing expectations (12s-30min), executable PATH requirements, platform-specific compiler needs

Key Context for Copilot

The dual architecture is critical - files with *GUI.py suffix require display/X11 and wxPython, while core modules work headless. Fortran extensions in sources/ require full rebuild via meson/f2py. Tests need LATTIC and convcell executables in PATH.

File structure now: 324 lines, 12 major sections, 32 code examples.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • gh.io
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>✨ Set up Copilot instructions</issue_title>
<issue_description>Configure instructions for this repository as documented in Best practices for Copilot coding agent in your repository.

</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…d best practices

Co-authored-by: briantoby <9433704+briantoby@users.noreply.github.com>
Copilot AI changed the title [WIP] Add configuration instructions for Copilot coding agent Enhance Copilot instructions with architecture, style, and development context Jan 28, 2026
Copilot AI requested a review from briantoby January 28, 2026 20:29
Removed linting and formatting section for ruff from instructions.
Copy link
Collaborator

@briantoby briantoby left a comment

Choose a reason for hiding this comment

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

looks good to me

@briantoby briantoby marked this pull request as ready for review January 28, 2026 20:57
@briantoby briantoby merged commit ee134b8 into main Jan 28, 2026
24 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.

✨ Set up Copilot instructions

2 participants