Skip to content

fix pep8 syntax check #333

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 12, 2023
Merged

fix pep8 syntax check #333

merged 1 commit into from
Jan 12, 2023

Conversation

g0rgamesh
Copy link
Contributor

No description provided.

@g0rgamesh g0rgamesh requested a review from a team as a code owner January 11, 2023 15:43
@g0rgamesh g0rgamesh requested a review from mnasiadka January 11, 2023 15:43
@jovial
Copy link
Contributor

jovial commented Jan 11, 2023 •
edited
Loading

I think we need this in xena too. See CI failure in #324. Best to re-target xena and sync up?

@g0rgamesh g0rgamesh merged commit a340aa2 into stackhpc/yoga Jan 12, 2023
@g0rgamesh g0rgamesh deleted the pep8_fix branch January 12, 2023 08:30
@markgoddard
Copy link
Contributor

Please target s-k-c changes to the earliest relevant branch. They will then get merged forwards.

@markgoddard
Copy link
Contributor

I've backported it to the xena/wallaby sync.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mnasiadka mnasiadka mnasiadka approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@g0rgamesh @jovial @markgoddard @mnasiadka