Skip to content

Add Copilot instructions#76

Merged
Oleg-Melnik merged 4 commits into
masterfrom
add-copilot-config
Jun 26, 2026
Merged

Add Copilot instructions#76
Oleg-Melnik merged 4 commits into
masterfrom
add-copilot-config

Conversation

@Oleg-Melnik

Copy link
Copy Markdown
Collaborator

This PR adds Copilot instructions.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new .github/copilot-instructions.md file to provide repository-specific guidance for Copilot/agents, including project layout, review focus, and suggested verification commands.

Changes:

  • Introduces a comprehensive Copilot instructions document for repo context, code areas, and review priorities.
  • Documents expected testing/linting/verifications commands aligned with CI.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/copilot-instructions.md Outdated
Comment thread .github/copilot-instructions.md Outdated
Oleg-Melnik and others added 2 commits June 26, 2026 11:44
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

@Vladyslav-Kuksiuk Vladyslav-Kuksiuk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@Oleg-Melnik, LGTM, with minor comment.

However, all of this information is already mentioned in other agent-related documentation. Is this file intended to consolidate it in one place, even if it is a duplication?

Comment thread .github/copilot-instructions.md Outdated
Comment on lines +96 to +97
notes. Keep [`AGENTS.md`](../AGENTS.md) focused on repository-wide agent
policy.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Do not leave orphans.

This rule already exists in the writer skill. You can try to force the AI to this skill if, for some reason, it was not applied.

Comment thread .github/copilot-instructions.md Outdated
Run focused package tests when they prove the change:

```bash
go test -v ./cli -p 1

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

After this PR merged -p 1 is redundant.

Same in other places.

@Oleg-Melnik

Copy link
Copy Markdown
Collaborator Author

@codex[agent] review and check every comment is addressed.

@Oleg-Melnik Oleg-Melnik merged commit 0604985 into master Jun 26, 2026
4 checks passed
@Oleg-Melnik Oleg-Melnik deleted the add-copilot-config branch June 26, 2026 10:09

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2555fa8b29

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/copilot-instructions.md
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.

3 participants