Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tool state: wrong tool_config path loaded #18556

Closed
mvdbeek opened this issue Jul 17, 2024 · 2 comments · Fixed by #18646
Closed

Tool state: wrong tool_config path loaded #18556

mvdbeek opened this issue Jul 17, 2024 · 2 comments · Fixed by #18646

Comments

@mvdbeek
Copy link
Member

mvdbeek commented Jul 17, 2024

path_to_tool = os.path.join(work_dir, tool_config)
only works as intended if tool_config is not an absolute path, but it is an absolute path. I'm unsure if we should just strip the repo path here, or if we should reset this using a repository metadata reset.

@mvdbeek
Copy link
Member Author

mvdbeek commented Jul 17, 2024

Should be fixed in #18558

@mvdbeek mvdbeek closed this as completed Jul 17, 2024
@jmchilton jmchilton reopened this Aug 5, 2024
@jmchilton
Copy link
Member

Broken still after deployment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants