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 #3950,#3951: Added an new version of motdConfiguration enabling th... #191

Conversation

Kegeruneku
Copy link

...e user to select where should the MOTD be added

Ticket: http://www.rudder-project.org/redmine/issues/3951

To be merged in 2.4

###################################################

bundle edit_line prepend(lines)
bundle edit_line rudder_line_insertion(lines, location)
Copy link
Member

Choose a reason for hiding this comment

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

We should never, ever, rename bodies or bundles in shared libraries. There may be many Techniques out there using them.

Just add a new bundle/body if you need to.

@jooooooon
Copy link
Member

This contains unrelated changes in initial promises and other Techniques. Aside from them, and the one comment I made about a description this looks fine.

Please clean up the other changes, and we're good to go!

@Kegeruneku
Copy link
Author

Updated !

… the user to select where should the MOTD be added
jooooooon added a commit that referenced this pull request Sep 23, 2013
…end_start_option_to_motdconfiguration

Fixes #3950,#3951: Added an new version of motdConfiguration enabling th...
@jooooooon jooooooon merged commit 713f5ba into Normation:branches/rudder/2.4 Sep 23, 2013
amousset pushed a commit to amousset/rudder-techniques that referenced this pull request Dec 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants