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

Fixes #32539 - Stop loading rex core into regular rex #586

Merged
merged 1 commit into from May 10, 2021

Conversation

adamruzicka
Copy link
Contributor

No description provided.

@ezr-ondrej
Copy link
Member

This is really all what it takes to get rid of this? 🤦

@adamruzicka
Copy link
Contributor Author

Well, not really. It just prepares the ground for this one #587

@ezr-ondrej
Copy link
Member

I meant to get rid of the dependency of rex to rex_core :) I always thought there is something we really share between proxy and foreman, that I still just don't understand

@adamruzicka
Copy link
Contributor Author

Ah, there used to be. But now this seems to be the only thing left.

Copy link
Member

@ezr-ondrej ezr-ondrej left a comment

Choose a reason for hiding this comment

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

Could we drop the dependency from rex gemspec now then, or do I misunderstand? :)

@adamruzicka
Copy link
Contributor Author

I would like to keep that around until we remove rex_core for good so we don't mess up the dependency chain. I'm not sure if anything pulls in rex core explicitly.

@ezr-ondrej ezr-ondrej merged commit 92d6b2e into theforeman:master May 10, 2021
@adamruzicka adamruzicka deleted the nospdc branch May 10, 2021 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants