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

[Review] Request from 'schubi2' @ 'yast/yast-autoinstallation/review_160113_bug_955878_merged_with_sp1' #179

Closed
wants to merge 5 commits into from

Conversation

Projects
None yet
3 participants
@schubi2
Copy link
Member

commented Jan 13, 2016

Please review the following changes:

schubi2 added some commits Jan 13, 2016

@kobliha

This comment has been minimized.

Copy link
Member

commented Jan 13, 2016

I'm afraid that this is wrong as well as the previous PR was. We want to ignore cobbler completely.
We don't want to show error or warning on this. But this is just my POV, maybe customers/PM/PjM want something different.

@schubi2

This comment has been minimized.

Copy link
Member Author

commented Jan 14, 2016

I think espl. in Master we should go that way in order to get rid of the unused cobbler entry in the future .

@mvidner

This comment has been minimized.

Copy link
Member

commented Mar 18, 2016

This was replaced by #181.

@mvidner mvidner closed this Mar 18, 2016

@schubi2 schubi2 deleted the review_160113_bug_955878_merged_with_sp1 branch Aug 17, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.