Skip to content

Conversation

Copy link

Copilot AI commented Feb 9, 2026

arc/job/adapter_test.py references ARC_PATH on line 211 but was only importing ARC_TESTING_PATH, causing CI test failures.

Changes:

  • Added ARC_PATH to the import statement from arc.common
# Before
from arc.common import ARC_TESTING_PATH

# After  
from arc.common import ARC_PATH, ARC_TESTING_PATH
Original prompt

Pull Request: #807

Fix the error in the ci where it was missing the import of ARC_PATH


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: calvinp0 <56573970+calvinp0@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix error in CI due to missing ARC_PATH import Fix missing ARC_PATH import in adapter_test.py Feb 9, 2026
Copilot AI requested a review from calvinp0 February 9, 2026 17:28
@calvinp0 calvinp0 marked this pull request as ready for review February 9, 2026 17:42
Copilot AI review requested due to automatic review settings February 9, 2026 17:42
@calvinp0 calvinp0 merged commit 4ce45bc into crest_adapter Feb 9, 2026
4 checks passed
@calvinp0 calvinp0 deleted the copilot/fix-ci-arc-path-import branch February 9, 2026 17:42
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

Fixes a CI/unit-test failure in ARC’s job adapter tests by importing a missing path constant used to locate a test fixture.

Changes:

  • Update arc/job/adapter_test.py to import ARC_PATH alongside ARC_TESTING_PATH from arc.common.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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