Feature #1402

multi-line comments

Added by Lawrence Ludwig over 3 years ago. Updated almost 2 years ago.

Status:Closed Start date:07/08/2008
Priority:Low Due date:
Assignee:James Turnbull % Done:

0%
Category:language
Target version:0.24.7
Affected Puppet version:0.24.4 Branch:
Keywords:comment multiline
Votes: 0

Description

I could really use multi-line comments when developing/testing code. Is it possible than just the # (shell style) option?

History

#1
Updated by Lawrence Ludwig over 3 years ago

lludwig wrote:

I could really use multi-line comments when developing/testing code. Is it possible than just the # (shell style) option?

IE.

/*

some puppet code I don't want to run

*/

#2
Updated by James Turnbull over 3 years ago

  • Status changed from Unreviewed to Accepted
  • Assignee set to Luke Kanies
  • Target version set to 4
  • Keywords set to comment multiline

#3
Updated by James Turnbull over 3 years ago

  • Category set to language

#4
Updated by Brice Figureau over 3 years ago

lludwig wrote:

I could really use multi-line comments when developing/testing code. Is it possible than just the # (shell style) option?

I fixed this issue in the tickets/0.24.x/1402 branch of my github repository: http://github.com/masterzen/puppet/tree/tickets%2F0.24.x%2F1402

I chose the C/C++ syntax / / which is a de-facto standard nowadays.

#5
Updated by Luke Kanies over 3 years ago

  • Status changed from Accepted to Ready For Checkin
  • Assignee changed from Luke Kanies to James Turnbull

#6
Updated by James Turnbull over 3 years ago

  • Status changed from Ready For Checkin to Tests Insufficient
  • Assignee changed from James Turnbull to Brice Figureau

Brice – can you please rebase this off the current 0.24.x please.

#7
Updated by Brice Figureau over 3 years ago

  • Status changed from Tests Insufficient to Ready For Checkin
  • Assignee changed from Brice Figureau to James Turnbull

Hi James,

It is now freshly rebased. And still in the tickets/0.24.x/1402 branch of my github repository: http://github.com/masterzen/puppet/tree/tickets%2F0.24.x%2F1402

#8
Updated by James Turnbull over 3 years ago

  • Status changed from Ready For Checkin to Code Insufficient

#9
Updated by Brice Figureau over 3 years ago

  • Status changed from Code Insufficient to Ready For Checkin

Hi James,

I rebased both #857 and #1402 on the same branch (tickets/0.24.x/1402+857) to limit the risk of conflicts, as discussed on #puppet: http://github.com/masterzen/puppet/tree/tickets%2F0.24.x%2F1402+857

#10
Updated by James Turnbull over 3 years ago

  • Status changed from Ready For Checkin to Closed

Pushed in commit:2d37f09aa093b10cb64b9b649f0066217c53d48f in branch 0.24.x

#11
Updated by James Turnbull over 2 years ago

  • Target version changed from 4 to 0.24.7

Also available in: Atom PDF