Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove dependency cycle caused by addition of foreman-service anchor #156

Merged
merged 1 commit into from
Nov 30, 2016

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Nov 20, 2016

No description provided.

@ehelms
Copy link
Member Author

ehelms commented Nov 20, 2016

Still fully testing this but this will require all the changes associated to: Katello/katello#6465

@ekohl
Copy link
Member

ekohl commented Nov 20, 2016

Will this continue working?

@ehelms ehelms force-pushed the fix-seed branch 9 times, most recently from 2d2fc3b to 6f67e3c Compare November 23, 2016 18:03
@ehelms ehelms force-pushed the fix-seed branch 7 times, most recently from 5971cff to 6756c0a Compare November 29, 2016 20:57
Copy link
Member

@stbenjam stbenjam left a comment

Choose a reason for hiding this comment

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

Tested and works for me

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

Yay better dependencies

@ehelms ehelms merged commit 5b4dd28 into theforeman:master Nov 30, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants