Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

fix(job_key): changed job_id to job_key#559

Merged
giuliastf merged 1 commit intomainUiPath/uipath-python:mainfrom
fix/job_keyUiPath/uipath-python:fix/job_keyCopy head branch name to clipboard
Sep 15, 2025
Merged

fix(job_key): changed job_id to job_key#559
giuliastf merged 1 commit intomainUiPath/uipath-python:mainfrom
fix/job_keyUiPath/uipath-python:fix/job_keyCopy head branch name to clipboard

Conversation

@giuliastf
Copy link
Collaborator

@giuliastf giuliastf commented Sep 10, 2025 •
edited by github-actions bot
Loading

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
Copilot AI reviewed Sep 10, 2025
View reviewed changes
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

Reviewers

Copilot code review Copilot Copilot left review comments

@cristipufu cristipufu cristipufu approved these changes

@radu-mocanu radu-mocanu radu-mocanu approved these changes

Assignees

@giuliastf giuliastf

Labels

build:dev Create a dev build from the pr

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Process Invoke should send the parent job key

3 participants

@giuliastf @cristipufu @radu-mocanu
Morty Proxy This is a proxified and sanitized view of the page, visit original site.