google / cel-spec

Common Expression Language -- specification and binary representation
https://cel.dev
Apache License 2.0
2.78k stars 222 forks source link

Timezone format inconsistent with conformance tests #194

Closed jsannemo closed 3 years ago

jsannemo commented 3 years ago

The spec states that time zones are

TimeZone = "UTC" | LongTZ | FixedTZ ;
LongTZ = ? list available at
           http://joda-time.sourceforge.net/timezones.html ? ;
FixedTZ = ( "+" | "-" ) Digit Digit ":" Digit Digit ;
Digit = "0" | "1" | ... | "9" ;

but the conformance tests includes several time zones that don't conform do this, such as

Personally I think the spec does this right, and the conformance tests is too generous. Either way, they should be the same.