Skip to content

Fixup some un-needed string interpolations#671

Merged
aglarendil merged 1 commit into
Mirantis:masterfrom
xarses:fix-some-quotes
Nov 5, 2013
Merged

Fixup some un-needed string interpolations#671
aglarendil merged 1 commit into
Mirantis:masterfrom
xarses:fix-some-quotes

Conversation

@xarses

@xarses xarses commented Oct 9, 2013

Copy link
Copy Markdown
Contributor

Removes some unnecessary string interpolations and adds line breaks

Fix horizon string interpolation
Fix l23network string interpolation
Fix quantum string interpolation

@mirantis-jenkins

Copy link
Copy Markdown

👍 Test PASSed.
Refer to this link for build results: http://fuel-jenkins.mirantis.com/job/fuel_syntax_check/350/

@mihgen

mihgen commented Oct 10, 2013

Copy link
Copy Markdown
Contributor

+1. Passes CentOS deployment of controller,compute,cinder on 129th ISO

@aglarendil

Copy link
Copy Markdown

this is just simple lighweight fix. no need to merge it in this release

@dmitryilyin

Copy link
Copy Markdown
Contributor

Looks nice but will become PITA for those who will rebse their branches on top of it.

@mihgen

mihgen commented Oct 24, 2013

Copy link
Copy Markdown
Contributor

rebase needed

Fix horizon string interpolation
Fix quantum string interpolation
@mirantis-jenkins

Copy link
Copy Markdown

Merged build triggered.

@mirantis-jenkins

Copy link
Copy Markdown

Merged build started.

@aglarendil

Copy link
Copy Markdown

+2

aglarendil pushed a commit that referenced this pull request Nov 5, 2013
Fixup some un-needed string interpolations
@aglarendil aglarendil merged commit df58134 into Mirantis:master Nov 5, 2013
@xarses xarses deleted the fix-some-quotes branch November 5, 2013 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants