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 #16282: Explicitly require i18n, components and ui.router modules #6276

Merged
merged 1 commit into from Aug 26, 2016

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Aug 25, 2016

Changes in Bastion required explicit requires on some modules
that were detected by breaking tests.

@theforeman-bot
Copy link

There were the following issues with the commit message:

  • length of the first commit message line for c5d439455112c4a16863b138e400458033c4f5ca exceeds 65 characters

If you don't have a ticket number, please create an issue in Redmine, selecting the appropriate project.

More guidelines are available in Coding Standards or on the Foreman wiki.


This message was auto-generated by Foreman's prprocessor

@ehelms
Copy link
Member Author

ehelms commented Aug 25, 2016

Will require Katello/bastion#125 to fully pass

Changes in Bastion required explicit requires on some modules
that were detected by breaking tests.
@waldenraines
Copy link
Contributor

Hah, I was making the exact same changes so ACK pending other PR merge and Jenkins.

@ehelms
Copy link
Member Author

ehelms commented Aug 25, 2016

@waldenraines
Copy link
Contributor

ACK.

@waldenraines
Copy link
Contributor

I'm going to go ahead and merge this so we can have a green build in the morning.

@waldenraines waldenraines merged commit 7de4a11 into Katello:master Aug 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants