'str' in config.yaml should be renamed to 'string'
Bug #837708 reported by
Gustavo Niemeyer
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
pyjuju |
Fix Released
|
Undecided
|
Gustavo Niemeyer | ||
Juju Charms Collection |
Fix Released
|
High
|
Unassigned |
Bug Description
There's no value in saving the 3 chars when we have e.g. 'float' already in the same context, and 'string' is actually readable while 'str' is not.
Related branches
lp://qastaging/~niemeyer/pyjuju/no-regex-option
- Kapil Thangavelu (community): Approve
- William Reade (community): Approve
-
Diff: 267 lines (+85/-62)6 files modifiedensemble/formula/config.py (+31/-27)
ensemble/formula/tests/repository/dummy/config.yaml (+3/-3)
ensemble/formula/tests/repository/wordpress/config.yaml (+1/-1)
ensemble/formula/tests/test_config.py (+48/-29)
examples/php/config.yaml (+1/-1)
examples/wordpress/config.yaml (+1/-1)
description: | updated |
Changed in ensemble: | |
status: | New → In Progress |
assignee: | nobody → Gustavo Niemeyer (niemeyer) |
milestone: | none → eureka |
To post a comment you must log in.
Excerpts from Gustavo Niemeyer's message of Tue Aug 30 21:30:26 UTC 2011:
> Public bug reported:
>
> There's no value in saving the 3 chars when we have e.g. 'float'
> already in the same context, and 'string' is actually readable while
> 'str' is not.
Will you be keeping str for backward compatibility?
There are 25 official principia formulas.. not sure how many will need
to be updated for this change.