Skip to content

Add work directory to mache.deploy's check_call()#392

Merged
xylar merged 3 commits intoE3SM-Project:mainfrom
xylar:better-output-from-deploy
Apr 10, 2026
Merged

Add work directory to mache.deploy's check_call()#392
xylar merged 3 commits intoE3SM-Project:mainfrom
xylar:better-output-from-deploy

Conversation

@xylar
Copy link
Copy Markdown
Collaborator

@xylar xylar commented Apr 10, 2026 •
edited
Loading

This is needed for better debugging when mache.deploy is not behaving.

This PR also tries to improve AI agents' use of pre-commit.

Checklist

  • Tests pass and new features are covered by tests
  • Testing comment, if appropriate, in the PR documents testing used to verify the changes

@xylar xylar added the clean-up Cleanup or maintenance of code that does not alter behavior or functionality label Apr 10, 2026
@xylar
Write out context in json files in deploy_tmp.
7bc7068 
This should help with debugging.
@xylar
Copy link
Copy Markdown
Collaborator Author

xylar commented Apr 10, 2026

Testing

In deployments of cime-env, I'm seeing the work directories as expected. I have seen improved behavior in an AI agent after the more forceful push toward pre-commit. The json files are appearing as expected and have helped with debuging.

@xylar xylar merged commit 4899501 into E3SM-Project:main Apr 10, 2026
9 checks passed
@xylar xylar deleted the better-output-from-deploy branch April 10, 2026 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

clean-up Cleanup or maintenance of code that does not alter behavior or functionality

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@xylar