Skip to content

fix(job_key): changed job_id to job_key#559

Merged
giuliastf merged 1 commit intomainfrom
fix/job_key
Sep 15, 2025
Merged

fix(job_key): changed job_id to job_key#559
giuliastf merged 1 commit intomainfrom
fix/job_key

Conversation

@giuliastf
Copy link
Collaborator

@giuliastf giuliastf commented Sep 10, 2025

Fix: #539

Development Package

  • Add this package as a dependency in your pyproject.toml:
[project]
dependencies = [
  # Exact version:
  "uipath==2.1.44.dev1005591056",

  # Any version from PR
  "uipath>=2.1.44.dev1005590000,<2.1.44.dev1005600000"
]

[[tool.uv.index]]
name = "testpypi"
url = "https://test.pypi.org/simple/"
publish-url = "https://test.pypi.org/legacy/"
explicit = true

[tool.uv.sources]
uipath = { index = "testpypi" }

@giuliastf giuliastf self-assigned this Sep 10, 2025
@giuliastf giuliastf requested a review from Copilot September 10, 2025 12:27
Copy link

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 fixes an inconsistency in environment variable naming by changing references from job_id to job_key to align with the actual header name being used.

  • Updated import to use ENV_JOB_KEY instead of ENV_JOB_ID
  • Changed environment variable lookup to use the correct ENV_JOB_KEY constant

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@giuliastf giuliastf added the build:dev Create a dev build from the pr label Sep 10, 2025
@giuliastf giuliastf enabled auto-merge September 15, 2025 10:52
@giuliastf giuliastf disabled auto-merge September 15, 2025 10:53
@giuliastf giuliastf merged commit 2cc1967 into main Sep 15, 2025
16 checks passed
@giuliastf giuliastf deleted the fix/job_key branch September 15, 2025 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build:dev Create a dev build from the pr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Process Invoke should send the parent job key

3 participants