Re: [PHP-DEV] [RFC][VOTE] Flexible Heredoc and Nowdoc Syntaxes (voting restarted)

[Stanislav Malyshev]

Hi!

> As per feedback here, I have updated the first voting question for
> additional clarification. This has restarted the vote for the first
> question only. The second vote will continue as normal, and both will
> still end on the same date (November 15th).

Thank you!

-- 
Stas Malyshev
smalys...@gmail.com

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Re: [PHP-DEV] [RFC][VOTE] Flexible Heredoc and Nowdoc Syntaxes (voting restarted)

[Peter Cowburn]

On 2 November 2017 at 11:35, Thomas Punt  wrote:

> Hi all,
>
>
> > I would add that this is particularly important on an RFC with two or
> more votes. On most RFCs, the voting question is implied to be "accept the
> change/feature as described above", but as soon as you have two votes, it's
> important to be clear which parts of the proposal are covered by each vote.
>
>
> As per feedback here, I have updated the first voting question for
> additional clarification. This has restarted the vote for the first
> question only. The second vote will continue as normal, and both will still
> end on the same date (November 15th).
>
>
Link to RFC: https://wiki.php.net/rfc/flexible_heredoc_nowdoc_syntaxes


>
> Sorry for the inconvenience.


>
> -Tom
>

Re: [PHP-DEV] [RFC][VOTE] Flexible Heredoc and Nowdoc Syntaxes (voting restarted)

[Thomas Punt]

Hi all,


> I would add that this is particularly important on an RFC with two or more 
> votes. On most RFCs, the voting question is implied to be "accept the 
> change/feature as described above", but as soon as you have two votes, it's 
> important to be clear which parts of the proposal are covered by each vote.


As per feedback here, I have updated the first voting question for additional 
clarification. This has restarted the vote for the first question only. The 
second vote will continue as normal, and both will still end on the same date 
(November 15th).


Sorry for the inconvenience.


-Tom

Re: [PHP-DEV] [RFC][VOTE] Flexible Heredoc and Nowdoc Syntaxes

[Rowan Collins]

On 2 November 2017 04:24:17 GMT+00:00, Stanislav Malyshev  
wrote:
>Hi!
>
>> From the body of the RFC:
>> 
>> "To enable for the closing marker to be indented, ... The indentation
>> of the closing marker dictates the amount of whitespace to strip from
>> each line within the heredoc/nowdoc"
>
>Yes, that's what the RFC says. But the voting question only mentions
>indenting the marker. I'd prefer voting options were clear and I didn't
>have to second-guess the intent. It could be "Allow indenting the
>marker
>and stripping the whitespace?" and all would be clear.

I would add that this is particularly important on an RFC with two or more 
votes. On most RFCs, the voting question is implied to be "accept the 
change/feature as described above", but as soon as you have two votes, it's 
important to be clear which parts of the proposal are covered by each vote.

Regards,

-- 
Rowan Collins
[IMSoP]

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Re: [PHP-DEV] [RFC][VOTE] Flexible Heredoc and Nowdoc Syntaxes

[Stanislav Malyshev]

Hi!

> From the body of the RFC:
> 
> "To enable for the closing marker to be indented, ... The indentation
> of the closing marker dictates the amount of whitespace to strip from
> each line within the heredoc/nowdoc"

Yes, that's what the RFC says. But the voting question only mentions
indenting the marker. I'd prefer voting options were clear and I didn't
have to second-guess the intent. It could be "Allow indenting the marker
and stripping the whitespace?" and all would be clear.

> "RFC voters - please read the RFC during the RFC period and ask
> questions then. ;-)" is an irritating way of writing this.

I've read the RFC, that is exactly why I was asking - because the voting
question did not match the text of the RFC, omitting rather serious side
effect.

-- 
Stas Malyshev
smalys...@gmail.com

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Re: [PHP-DEV] [RFC][VOTE] Flexible Heredoc and Nowdoc Syntaxes

[Dan Ackroyd]

Stanislav Malyshev  wrote:
>
> Something I am missing here. RFC is talking about removing indents from
> heredoc text, but the vote says "Allow for the closing marker to be
> indented?" and does not mention that indenting closing marker also
> changes how the rest of the heredoc is parsed. Is this still the case?


>From the body of the RFC:

"To enable for the closing marker to be indented, ... The indentation
of the closing marker dictates the amount of whitespace to strip from
each line within the heredoc/nowdoc"

That's about 3 lines of text into the RFC. Presumably the RFC didn't
think the whole concept being voted on needed to be repeated in the
voting option. Perhaps it should have been but:

Christopher Jones wrote:
> I'm going to have to assume so and vote No.

It is better to ask questions during the discussion phase of an RFC,
but if you do have a question during the voting period, you're still
allowed to ask rather than voting based on assumptions.

> PS. RFC writers: please take care when writing specs.

Please don't use passive aggressive 'helpful hints' like this. For example:

"RFC voters - please read the RFC during the RFC period and ask
questions then. ;-)" is an irritating way of writing this.

A way of writing it that is more conducive to a productive discussion is:

I think this is covered in the RFC text. Does the line I picked out
above, give a clear description for what that vote is for?

cheers
Dan
Ack

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Re: [PHP-DEV] [RFC][VOTE] Flexible Heredoc and Nowdoc Syntaxes

[Christopher Jones]

On 2/11/17 8:58 am, Stanislav Malyshev wrote:

Hi!


Voting has now started on the flexible heredoc and nowdoc syntaxes RFC[1].


Voting will be open for 2 weeks (until November 15th).

Something I am missing here. RFC is talking about removing indents from
heredoc text, but the vote says "Allow for the closing marker to be
indented?" and does not mention that indenting closing marker also
changes how the rest of the heredoc is parsed. Is this still the case?




I'm going to have to assume so and vote No.

Chris

PS. RFC writers: please take care when writing specs.


--
http://twitter.com/ghrd


--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Re: [PHP-DEV] [RFC][VOTE] Flexible Heredoc and Nowdoc Syntaxes

[Stanislav Malyshev]

Hi!

> Voting has now started on the flexible heredoc and nowdoc syntaxes RFC[1].
> 
> 
> Voting will be open for 2 weeks (until November 15th).

Something I am missing here. RFC is talking about removing indents from
heredoc text, but the vote says "Allow for the closing marker to be
indented?" and does not mention that indenting closing marker also
changes how the rest of the heredoc is parsed. Is this still the case?



-- 
Stas Malyshev
smalys...@gmail.com

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-DEV] [RFC][VOTE] Flexible Heredoc and Nowdoc Syntaxes

[Thomas Punt]

Hi internals!


Voting has now started on the flexible heredoc and nowdoc syntaxes RFC[1].


Voting will be open for 2 weeks (until November 15th).


Thanks,

Tom


[1]: https://wiki.php.net/rfc/flexible_heredoc_nowdoc_syntaxes