Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 23, 2025

Changes to engineering infrastructure (pipeline configs, build scripts, rulesets) in packages/http-client-csharp/eng/ should not trigger emitter version publishes.

Changes

Modified packages/http-client-csharp/eng/pipeline/publish.yml to exclude the eng/ directory from publish triggers:

trigger:
  branches:
    include:
      - main
  paths:
    include:
      - packages/http-client-csharp/
    exclude:
      - packages/http-client-csharp/eng/

This prevents unnecessary publishes when modifying:

  • Pipeline YAML files
  • PowerShell build/test scripts
  • Code analysis rulesets and StyleCop configs
  • Pipeline images and other eng resources
Original prompt

This section details on the original issue you should resolve

<issue_title>Don't publish new emitter version if only changes are to eng directory</issue_title>
<issue_description>We should exclude changes to http-client-csharp/eng path as triggering a publish.</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI self-assigned this Nov 23, 2025
Copilot AI changed the title [WIP] Fix publish trigger for emitter version Exclude eng/ directory from http-client-csharp publish triggers Nov 23, 2025
Copilot AI requested a review from JoshLove-msft November 23, 2025 19:14
Copilot finished work on behalf of JoshLove-msft November 23, 2025 19:14
@JoshLove-msft JoshLove-msft marked this pull request as ready for review November 24, 2025 03:15
@JoshLove-msft JoshLove-msft added this pull request to the merge queue Nov 25, 2025
Merged via the queue into main with commit 1530d5a Nov 25, 2025
22 checks passed
@JoshLove-msft JoshLove-msft deleted the copilot/exclude-eng-directory-changes branch November 25, 2025 22:31
@microsoft-github-policy-service microsoft-github-policy-service bot added the emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp label Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Don't publish new emitter version if only changes are to eng directory

3 participants