Skip to content

[DO NOT MERGE] Copy of #2649 - #2734

Closed
Xiaoyu Z (xiaoyu-work) wants to merge 3 commits into
mainfrom
fix/run-pass-action-decorator
Closed

Xiaoyu Z (xiaoyu-work) wants to merge 3 commits into
mainfrom
fix/run-pass-action-decorator

Conversation

@xiaoyu-work

Copy link
Copy Markdown
Member

Describe your changes

DO NOT MERGE. Independent draft copy of #2649.

Copies sylvesterkaczmarek/Olive branch fix/run-pass-action-decorator to the destination-owned microsoft/Olive branch of the same name. Original commits are preserved unchanged at 66fad87dd6c4318cf4060dd7740383a66007c32c.

The original change moves the telemetry @action decorator from the RunPassCommand class to its run() method, recording command execution rather than construction. It includes regression coverage for the class and decorated method.

Tests and lintrunner -a were not run; this operation only copies the existing source branch.

Checklist before requesting a review

  • Add unit tests for this change.
  • Make sure all tests can pass.
  • Update documents if necessary.
  • Lint and apply fixes to your code by running lintrunner -a
  • Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

Original PR: #2649.

Related: #2516.

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