Skip to content

Comments

Add Copilot coding agent instructions#10

Merged
alexlib merged 2 commits intomasterfrom
copilot/setup-copilot-instructions
Feb 19, 2026
Merged

Add Copilot coding agent instructions#10
alexlib merged 2 commits intomasterfrom
copilot/setup-copilot-instructions

Conversation

Copy link
Contributor

Copilot AI commented Feb 19, 2026

Adds .github/copilot-instructions.md to give GitHub Copilot structured context about this repository so it can assist more accurately.

What's included

  • Repo layout – maps src/, easysba/, tests/, and config files to their roles
  • Build requirements – Python 3.11+, C toolchain, BLAS/LAPACK, install commands
  • Test instructionspytest invocation and the EASYSBA_RUN_SOLVER=1 flag required to run the solver tests
  • Conventions – PEP 8 for Python, match existing style for C/C++, unittest.TestCase + pytest discovery
  • Key files table – quick-reference mapping of the most important source files to their purpose

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)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: alexlib <747110+alexlib@users.noreply.github.com>
Copilot AI changed the title [WIP] Configure Copilot instructions for the repository Add Copilot coding agent instructions Feb 19, 2026
Copilot AI requested a review from alexlib February 19, 2026 17:10
@alexlib alexlib marked this pull request as ready for review February 19, 2026 17:20
Copilot AI review requested due to automatic review settings February 19, 2026 17:20
@alexlib alexlib merged commit b8fd93a into master Feb 19, 2026
3 checks passed
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds .github/copilot-instructions.md to provide structured context about the easySBA Python bindings repository for GitHub Copilot. The instructions document repository layout, build requirements, testing procedures, coding conventions, and key files to help Copilot assist more accurately with code generation and review tasks.

Changes:

  • Added comprehensive Copilot instructions file documenting repository structure, build/test procedures, and coding conventions
  • Included a key files reference table mapping important source files to their purposes
  • Documented the EASYSBA_RUN_SOLVER environment variable requirement for solver tests


- Python code follows standard PEP 8 conventions.
- C/C++ code in `src/` and `easysba/_easysba.cpp` follows the style of the existing sources.
- Tests live in `tests/` and use `unittest.TestCase` classes, discovered and run via `pytest`.
Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

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

The statement "Tests live in tests/ and use unittest.TestCase classes" is not entirely accurate. While test_smoke.py uses unittest.TestCase, test_sba_steps.py uses plain pytest functions with pytest decorators like @pytest.mark.parametrize. Consider updating this to reflect both testing styles: "Tests live in tests/ and use either unittest.TestCase classes or plain pytest functions, discovered and run via pytest."

Suggested change
- Tests live in `tests/` and use `unittest.TestCase` classes, discovered and run via `pytest`.
- Tests live in `tests/` and use either `unittest.TestCase` classes or plain pytest functions, discovered and run via `pytest`.

Copilot uses AI. Check for mistakes.
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