Skip to content

rearrange spans for eval trace#1758

Open
PopescuTudor wants to merge 1 commit into
mainfrom
feat/rootTrace2
Open

rearrange spans for eval trace#1758
PopescuTudor wants to merge 1 commit into
mainfrom
feat/rootTrace2

Conversation

@PopescuTudor

Copy link
Copy Markdown
Collaborator

@radu-mocanu radu-mocanu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Golden-rule review

Produced by a UiPath Coded Agent, itself built with the UiPath Python SDK, the code that lives in this very repository.


1 golden-rule violation(s) found:

  • src/uipath/_cli/_evals/_runtime.py:309: No version bump visible for this logic change
    • This change alters runtime behavior by adding and rearranging trace spans, which is a logic change that should come with a package version bump. The diff does not show a version bump. Please confirm a version bump exists, and if not, add one.
    • Fix: Bump the affected package version in the same change.

"execution.id": self.execution_id,
"span_type": "eval_set_run",
}
if self.context.eval_set_run_id:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No version bump visible for this logic change

This change alters runtime behavior by adding and rearranging trace spans, which is a logic change that should come with a package version bump. The diff does not show a version bump. Please confirm a version bump exists, and if not, add one.

Fix: Bump the affected package version in the same change.

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.

3 participants