Skip to content

Added copilot instructions and claude action#677

Merged
VisLab merged 4 commits intohed-standard:mainfrom
VisLab:update_hed_cache
Mar 18, 2026
Merged

Added copilot instructions and claude action#677
VisLab merged 4 commits intohed-standard:mainfrom
VisLab:update_hed_cache

Conversation

@VisLab
Copy link
Member

@VisLab VisLab commented Mar 18, 2026

No description provided.

@VisLab VisLab requested a review from Copilot March 18, 2026 21:52
Copy link
Contributor

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 PR adds repository automation/configuration for contributor guidance and CI tooling, including new Copilot instructions, a Claude Code GitHub Action workflow, and a switch in spell-check tooling.

Changes:

  • Replace the Codespell workflow with a Typos-based spell-check workflow and add a .typos.toml configuration.
  • Add a Claude Code workflow that responds to @claude triggers on issues/PR comments/reviews (for permitted author associations).
  • Add/adjust repository hygiene/config files: .github/copilot-instructions.md, .gitattributes, and .gitignore updates (e.g., ignore .status/, .vscode/).

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
.typos.toml Adds Typos config (exclusions + allowlisted words) to support the new spelling CI check.
.gitignore Stops ignoring Copilot instructions; adds ignores for .status/ and .vscode/.
.github/workflows/typos.yaml Introduces a Typos spell-check workflow to replace Codespell.
.github/workflows/codespell.yaml Removes the previous Codespell workflow.
.github/workflows/claude.yaml Adds Claude Code automation triggered by @claude in approved contexts.
.github/copilot-instructions.md Adds repo-specific contributor/assistant instructions, commands, and CI checklist.
.gitattributes Adds LF normalization and binary/text handling rules across common file types.

💡 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.

@@ -0,0 +1,150 @@
# hed-javascript Copilot instructions
Comment on lines +139 to +142
| `codeql.yml` | automatic | CodeQL security scan (JavaScript) |
| `codespell.yaml` | automatic | Spell check source files |
| `links.yml` | automatic | Lychee link checker (configured via `lychee.toml`) |

Comment on lines +1 to +29
---
name: Typos

on:
push:
branches: [main]
pull_request:
branches: [main]

permissions:
contents: read

jobs:
typos:
name: Check for spelling errors
runs-on: ubuntu-latest

steps:
- name: Checkout
uses: actions/checkout@v6

- name: Install uv
uses: astral-sh/setup-uv@v7
with:
enable-cache: true
cache-dependency-glob: '.typos.toml'

- name: Run typos
run: uvx 'typos>=1.29.0'
@VisLab VisLab merged commit 0121b55 into hed-standard:main Mar 18, 2026
20 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