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

Switch to using enginefacade #175

Closed
wants to merge 1 commit into from
Closed

Switch to using enginefacade #175

wants to merge 1 commit into from

Conversation

scrungus
Copy link

Closes-Bug: #2067345
Change-Id: If9a2c96628cfcb819fee5e19f872ea015979b30f

Co-authored-by: scrungus [email protected]

Closes-Bug: #2067345
Change-Id: If9a2c96628cfcb819fee5e19f872ea015979b30f

Co-authored-by: scrungus <[email protected]>
@scrungus scrungus requested a review from a team as a code owner October 14, 2024 19:16
@mnasiadka
Copy link
Member

Can we get that at least upstream CI tested before we build that and deploy anywhere?

@mnasiadka
Copy link
Member

Oh boy, we don't even run tox tests in that repo.

@mnasiadka
Copy link
Member

Ah, because the PR is targeted for master...
@scrungus please target that for stackhpc/2024.1 repo.

@scrungus scrungus closed this Oct 17, 2024
@scrungus scrungus deleted the bugfix-2067345 branch October 17, 2024 11:34
@scrungus
Copy link
Author

closing in favour of #176

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.

3 participants