Skip to content

Ensure copied Mockly skill content is ignored in consuming repos#141

Merged
dennisdoomen merged 1 commit into
mainfrom
copilot/add-gitignore-to-copied-skills
Jun 2, 2026
Merged

Ensure copied Mockly skill content is ignored in consuming repos#141
dennisdoomen merged 1 commit into
mainfrom
copilot/add-gitignore-to-copied-skills

Conversation

Copilot AI commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

When Mockly copies its bundled skill into a consuming repository during build, the copied files can appear as untracked source-control noise. This change makes the generated skill directory self-ignoring by shipping and copying a .gitignore with the skill.

  • Package payload update

    • Added Mockly/skill.gitignore and packed it as skills/mockly/.gitignore in the NuGet package.
    • Updated Mockly.csproj package item definitions so the ignore file is distributed with the existing SKILL.md.
  • Build-transitive copy behavior

    • Extended Mockly.targets to copy .gitignore into .agents/skills/mockly alongside SKILL.md.
    • Preserved existing repo-root resolution and opt-out behavior (MocklySkill=false).
  • Resulting generated ignore rule

    *
    !.gitignore

Copilot AI changed the title Copy skill .gitignore into consuming projects Ensure copied Mockly skill content is ignored in consuming repos Jun 2, 2026
Copilot AI requested a review from dennisdoomen June 2, 2026 12:11
@dennisdoomen dennisdoomen marked this pull request as ready for review June 2, 2026 12:52
@dennisdoomen dennisdoomen added the bug Something isn't working label Jun 2, 2026
@github-actions

github-actions Bot commented Jun 2, 2026

Copy link
Copy Markdown

Test Results

  3 files  ±0    3 suites  ±0   9s ⏱️ +3s
155 tests ±0  155 ✅ ±0  0 💤 ±0  0 ❌ ±0 
307 runs  ±0  307 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit e664a62. ± Comparison against base commit c27fdf7.

♻️ This comment has been updated with latest results.

@dennisdoomen dennisdoomen merged commit 44c4755 into main Jun 2, 2026
10 checks passed
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 26818785079

Coverage remained the same at 84.624%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 1322
Covered Lines: 1186
Line Coverage: 89.71%
Relevant Branches: 447
Covered Branches: 311
Branch Coverage: 69.57%
Branches in Coverage %: Yes
Coverage Strength: 283.12 hits per line

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants