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

Templates for OpenStack Yoga version #61

Merged
merged 1 commit into from
Nov 8, 2024
Merged

Conversation

jouvin
Copy link
Contributor

@jouvin jouvin commented Nov 7, 2024

No description provided.

@jouvin jouvin added this to the 24.10 milestone Nov 7, 2024
@jouvin jouvin requested a review from jrha November 7, 2024 13:45
@jrha
Copy link
Member

jrha commented Nov 7, 2024

I presume the target branch for this one should be yoga not xena?

@jouvin jouvin force-pushed the yoga_version branch 3 times, most recently from d248fcb to 48e22d0 Compare November 7, 2024 14:47
@jouvin
Copy link
Contributor Author

jouvin commented Nov 7, 2024

@jrha some errors are left in panlint that I don't understand:

  • One too long line despite there is the comment line to ignore this line
  • A few errors reported that look as panlint bugs: complaints about - not surrounded by spaces in annotations...

@jouvin
Copy link
Contributor Author

jouvin commented Nov 7, 2024

I presume the target branch for this one should be yoga not xena?

Oh sheet... I fail to improve :-)

@jouvin jouvin changed the base branch from xena to yoga November 7, 2024 14:56
@jrha
Copy link
Member

jrha commented Nov 7, 2024

One too long line despite there is the comment line to ignore this line
A few errors reported that look as panlint bugs: complaints about - not surrounded by spaces in annotations...

Okay makes sense, the version of Panlint being run doesn't have these changes yet.

@jrha jrha merged commit 256e604 into quattor:yoga Nov 8, 2024
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants