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

Update repos.h, add Rocky Linux support, create nodes operational system presenter #11

Merged
merged 8 commits into from
May 25, 2023

Conversation

lbgracioso
Copy link
Collaborator

@lbgracioso lbgracioso commented May 19, 2023

I'm sorry about the amount of changed files but most of them have only small changes.

In this pull request I added the so expected support to Rocky Linux and also made few updates in the repos.h.

Edit: Create presenter to choose nodes operational system.

Copy link
Owner

@viniciusferrao viniciusferrao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lbgracioso I've reviewed some of your changes but unfortunately there's things that I don't understood correctly and there's some suggestions. I made some suggestions and comments that we can discuss.

Thanks.

src/repos.cpp Outdated Show resolved Hide resolved
src/repos.cpp Outdated Show resolved Hide resolved
src/repos.cpp Outdated Show resolved Hide resolved
src/services/shell.cpp Outdated Show resolved Hide resolved
@lbgracioso
Copy link
Collaborator Author

@viniciusferrao Please check the latest commit (27a91d5) .

…h dry-run variable to old value

Signed-off-by: lbgracioso <[email protected]>
@lbgracioso lbgracioso changed the title Update repos.h, add Rocky Linux support Update repos.h, add Rocky Linux support, create nodes operational system presenter May 25, 2023
@viniciusferrao viniciusferrao merged commit 16a954f into viniciusferrao:master May 25, 2023
@viniciusferrao
Copy link
Owner

Thanks @lbgracioso it's merged. The OS ISO selection section was very good.

@lbgracioso lbgracioso deleted the update_repos branch August 16, 2023 15:23
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.

2 participants