Skip to content

Updated the copilot structure and other tooling#675

Merged
VisLab merged 3 commits into
hed-standard:mainfrom
VisLab:add_instructions
Mar 20, 2026
Merged

Updated the copilot structure and other tooling#675
VisLab merged 3 commits into
hed-standard:mainfrom
VisLab:add_instructions

Conversation

@VisLab

@VisLab VisLab commented Mar 18, 2026

Copy link
Copy Markdown
Member

No description provided.

@VisLab VisLab requested a review from Copilot March 18, 2026 21:32

Copilot AI 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.

Pull request overview

This PR updates repository configuration and contributor tooling, including adding a committed Copilot instruction document and shifting spelling/link-check configuration to a new structure.

Changes:

  • Replace Codespell with typos (dependency + configuration + CI workflow update).
  • Consolidate Lychee ignore patterns into lychee.toml and delete .lycheeignore.
  • Add .github/copilot-instructions.md, introduce .gitattributes, and adjust .gitignore (stop ignoring Copilot instructions; ignore Ruff cache).

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
pyproject.toml Swap Codespell tooling for typos and add tool.typos configuration.
lychee.toml Expand and reformat link exclusions (absorbing former .lycheeignore entries).
.lycheeignore Remove standalone ignore file in favor of lychee.toml config.
.gitignore Stop ignoring Copilot instructions; add Ruff cache ignore.
.github/workflows/typos.yaml Update CI spelling job from Codespell to typos.
.github/copilot-instructions.md Add repository-specific Copilot/developer instructions and local/CI command guidance.
.gitattributes Normalize line endings to LF and mark common binary formats as binary.
Comments suppressed due to low confidence (2)

.github/workflows/typos.yaml:27

  • This workflow doesn't set python-version for astral-sh/setup-uv, while the other workflows do (e.g., mdformat/links use 3.12). Not pinning can make CI behavior drift when GitHub updates the runner defaults; consider setting an explicit Python version here for consistency and reproducibility.
    .github/workflows/typos.yaml:29
  • typos is now listed in .[dev], but the CI job runs it via uvx typos (which installs/runs an ephemeral tool version independent of your project dependency pins). Consider either (a) installing .[dev] and running uv run typos to ensure CI uses the same pinned version/config as contributors, or (b) dropping typos from .[dev] if uvx is the intended execution path.

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

You can also share your feedback on Copilot code review. Take the survey.

Comment thread .github/copilot-instructions.md
@VisLab VisLab changed the title Updated the copilot structure Updated the copilot structure and other tooling Mar 20, 2026
@VisLab VisLab merged commit 6b7bf7a into hed-standard:main Mar 20, 2026
4 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.

2 participants