Skip to content

ci: Fix docker build context#942

Merged
chtruong814 merged 1 commit intomainfrom
chtruong/fix-build
Aug 20, 2025
Merged

ci: Fix docker build context#942
chtruong814 merged 1 commit intomainfrom
chtruong/fix-build

Conversation

@chtruong814
Copy link
Contributor

What does this PR do ?

Fix docker build context

When the docker container builds, the source code is checked out in a folder path that uses the Run ID. Otherwise, it seems to fall back to the uv lock and pyproject from the main branch.

Issues

List issues that this PR closes (syntax):

Usage

  • You can potentially add a usage example below
# Add a code snippet demonstrating how to use this

Before your PR is "Ready for review"

Pre checks:

  • Make sure you read and followed Contributor guidelines
  • Did you write any new necessary tests?
  • Did you run the unit tests and functional tests locally? Visit our Testing Guide for how to run tests
  • Did you add or update any necessary documentation? Visit our Document Development Guide for how to write, build and test the docs.

Additional Information

  • ...

Signed-off-by: Charlie Truong <chtruong@nvidia.com>
@github-actions github-actions bot added the CI Relating to CI label Aug 20, 2025
@chtruong814 chtruong814 requested a review from terrykong August 20, 2025 00:11
@chtruong814 chtruong814 added the CI:docs Run doctest label Aug 20, 2025
@terrykong terrykong enabled auto-merge August 20, 2025 00:12
@terrykong terrykong added this pull request to the merge queue Aug 20, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 20, 2025
@chtruong814 chtruong814 added this pull request to the merge queue Aug 20, 2025
Merged via the queue into main with commit f85a4f6 Aug 20, 2025
34 checks passed
@chtruong814 chtruong814 deleted the chtruong/fix-build branch August 20, 2025 06:56
jveronvialard pushed a commit that referenced this pull request Aug 27, 2025
Signed-off-by: Charlie Truong <chtruong@nvidia.com>
Signed-off-by: Julien Veron Vialard <jveronvialar@nvidia.com>
soodoshll pushed a commit to soodoshll/RL that referenced this pull request Sep 4, 2025
Signed-off-by: Charlie Truong <chtruong@nvidia.com>
Signed-off-by: Qidong Su <qidongs@nvidia.com>
PrinsYin pushed a commit to PrinsYin/RL that referenced this pull request Nov 30, 2025
Signed-off-by: Charlie Truong <chtruong@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI:docs Run doctest CI Relating to CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants