Skip to content

Add GitHub Copilot Developer Skills for OVMS#4174

Open
rasapala wants to merge 1 commit intomainfrom
ovms_dev_skills
Open

Add GitHub Copilot Developer Skills for OVMS#4174
rasapala wants to merge 1 commit intomainfrom
ovms_dev_skills

Conversation

@rasapala
Copy link
Copy Markdown
Collaborator

@rasapala rasapala commented May 4, 2026

🛠 Summary

This PR introduces four GitHub Copilot skill files under .github/skills/ to streamline common developer workflows when working on the OpenVINO Model Server codebase.

New Skills

build-bazel-target
Triggers: bazel build, build ovms, //src:ovms
Purpose: Build any Bazel target inside the -build Docker container

build-builder-image
Triggers: make docker_build, rebuild -build image, make ovms_builder_image
Purpose: Build or rebuild the OVMS -build Docker image via the Makefile

run-single-gtest
Triggers: run this test, bazel test --test_filter, ovms_test
Purpose: Run or re-run a single gtest fixture/test case inside the container

run-make-style
Triggers: run make style, cpplint, clang-format-check, cppclean
Purpose: Run code-style and lint checks via make style

Details

  • Each skill file is a self-contained Markdown instruction document with YAML frontmatter describing when it should be invoked by Copilot.
  • Skills cover container reuse/creation, proxy forwarding (http_proxy/https_proxy/no_proxy), Red Hat distro support (--//:distro=redhat), and common error-reading steps (e.g. bazel-testlogs).
  • Skills include optional USER DEFAULTS placeholder blocks so individual developers can hardcode a preferred build container, target list, or distro without modifying the shared workflow logic.
  • Skills are auto-discovered by GitHub Copilot via YAML frontmatter in each file — no additional wiring is required. Copilot matches the skill to user requests based on the trigger phrases listed in each skill's description field.

Copy link
Copy Markdown
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 four GitHub Copilot “skill” instruction documents under .github/skills/ to standardize common OVMS developer workflows (Bazel builds/tests in the -build container, builder image rebuilds via Make, and host-side style checks).

Changes:

  • Add a skill for building arbitrary Bazel targets inside a user-specified -build Docker container.
  • Add a skill for rebuilding the OVMS -build Docker image via make ovms_builder_image / make docker_build with common flags and proxy handling.
  • Add skills for (a) running a single filtered C++ gtest via Bazel and (b) running make style and related lint/format targets.

Reviewed changes

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

File Description
.github/skills/build-bazel-target/SKILL.md Documents how to build OVMS Bazel targets inside a -build container, including proxy + Red Hat flag handling.
.github/skills/build-builder-image/SKILL.md Documents how to build/rebuild the -build Docker image via Make, with flags, tagging guidance, and validation steps.
.github/skills/run-make-style/SKILL.md Documents how to run make style and interpret/fix failures from spell/clang-format/cpplint/cppclean.
.github/skills/run-single-gtest/SKILL.md Documents running a single gtest (via --test_filter) inside a -build container, including log retrieval and proxy forwarding.

Comment thread .github/skills/build-builder-image/SKILL.md
Comment thread .github/skills/build-builder-image/SKILL.md Outdated
Comment thread .github/skills/build-bazel-target/SKILL.md Outdated
Comment thread .github/skills/run-single-gtest/SKILL.md Outdated
Comment thread .github/skills/build-bazel-target/SKILL.md
Comment thread .github/skills/build-builder-image/SKILL.md
Comment thread .github/skills/build-builder-image/SKILL.md
Copy link
Copy Markdown
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

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

Comment thread .github/skills/run-single-gtest/SKILL.md Outdated
Comment thread .github/skills/build-bazel-target/SKILL.md Outdated
Comment thread .github/skills/build-builder-image/SKILL.md Outdated
Comment thread .github/skills/run-make-style/SKILL.md Outdated
@rasapala rasapala force-pushed the ovms_dev_skills branch from 522e5f2 to ef83b07 Compare May 4, 2026 15:11
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