lp:~bac/charm-tools/1321316

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:charm-tools/1.2 at revision 322

charmers: Pending requested 2014-05-20

Diff: 46 lines (+16/-2)

2 files modified

charmtools/charms.py (+2/-2)
tests/test_charm_proof.py (+14/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1321316: proof dies if a 'type' is not specified for a config option

Undecided

Fix Released

Link a bug report

Related blueprints

324. By Brad Crittenden on 2014-05-20

Link to bug

323. By Brad Crittenden on 2014-05-20

.

322. By Brad Crittenden on 2014-05-20

Option types default to string without failing. Warning issued.

321. By Charles Butler on 2014-05-16

Tim Van Steenburgh 2014-05-16 Change handling of missing default values for config options.

320. By Charles Butler on 2014-05-16

Tim Van Steenburgh 2014-05-16 Fix 'charm proof' and 'make check'.
Tim Van Steenburgh 2014-05-15 Fix test_config.yaml parsing

319. By Marco Ceppi on 2014-05-01

[tvansteenburgh] Make 'charm proof' warn when a config option description is blank.

318. By <email address hidden> on 2014-04-28

[niedbalski] Fixes bug #1303793, helper now supports CNAME records lookups

317. By Brad Crittenden on 2014-04-08

Provide hidden options for proof server.

When doing QA for charmworld bundle proof changes to ensure they work with
charm-proof, it is troublesome to get charm-proof pointed at a local
charmworld instance or the one at staging.jujucharms.com.

This branch adds three new command-line option: --server, --port, and
--secure.

A typical use is:

% charm-proof -b --server=localhost --port=2464 -b .

or

% charm-proof -b --server=staging.jujucharms.com -b .

Note that 'help=argparse.SUPPRESS' has been used so that 'charm-proof -h' does
not show these options to the regular user. I think suppressing their
existence strikes the right balance between having options that only serve
developers while preserving the clean CLI for the average user.

R=Marco Ceppi
CC=
https://codereview.appspot.com/83700043

316. By Marco Ceppi on 2014-03-11

Sync

315. By Marco Ceppi on 2014-02-18

Fix deps, and minor version bump due to faulty upload