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

[WIP] Handle the %pre and %pre-install sections in the runtime module #5504

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

KKoukiou
Copy link
Contributor

@KKoukiou KKoukiou commented Mar 4, 2024

No description provided.

@pep8speaks
Copy link

pep8speaks commented Mar 4, 2024

Hello @KKoukiou! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 63:17: E201 whitespace after '{'
Line 63:31: E202 whitespace before '}'
Line 64:22: E201 whitespace after '{'
Line 64:36: E202 whitespace before '}'

Line 51:100: E501 line too long (136 > 99 characters)

Line 23:1: E302 expected 2 blank lines, found 1
Line 26:100: E501 line too long (107 > 99 characters)

Line 45:1: E302 expected 2 blank lines, found 1
Line 114:100: E501 line too long (123 > 99 characters)
Line 191:1: W391 blank line at end of file

Line 25:1: E302 expected 2 blank lines, found 1

Line 53:1: E302 expected 2 blank lines, found 1
Line 63:1: E302 expected 2 blank lines, found 1

Comment last updated at 2024-03-06 16:21:24 UTC

Implement the support for the %pre and %pre-install sections in the runtime module.
…tion

Allow to parse and generate %pre and %pre-install sections using a kickstart
specification. This solution will eventually replace the current
implementation.
Copy link

github-actions bot commented May 6, 2024

This PR is stale because it has been open 60 days with no activity.
Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the stale label May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation f41 infrastructure Changes affecting mostly infrastructure stale
2 participants