I forgot to publish my draft comments on Friday. I know everyone has been waiting patiently for them.
https://codereview.appspot.com/23740043/diff/1/charmworld/testing/factory.py File charmworld/testing/factory.py (right):
https://codereview.appspot.com/23740043/diff/1/charmworld/testing/factory.py#newcode365 charmworld/testing/factory.py:365: makeBundle = make_bundle On 2013/11/08 19:03:29, bac wrote: > You sure you'd not like to just fix them?
I'd like to... but I figured a mechanical branch later would be better.
https://codereview.appspot.com/23740043/diff/1/charmworld/views/api/__init__.py File charmworld/views/api/__init__.py (right):
https://codereview.appspot.com/23740043/diff/1/charmworld/views/api/__init__.py#newcode511 charmworld/views/api/__init__.py:511: bundle_metric_types = ('deployments') Good eye. Fixed.
https://codereview.appspot.com/23740043/diff/1/charmworld/views/api/__init__.py#newcode521 charmworld/views/api/__init__.py:521: if metric_name not in self.bundle_metric_types: On 2013/11/08 19:03:29, bac wrote: > Huh, it works anyway.
Well, I learned something new today.
https://codereview.appspot.com/23740043/diff/1/charmworld/views/tests/test_metrics_api.py File charmworld/views/tests/test_metrics_api.py (right):
https://codereview.appspot.com/23740043/diff/1/charmworld/views/tests/test_metrics_api.py#newcode2 charmworld/views/tests/test_metrics_api.py:2: On 2013/11/08 19:03:29, bac wrote: > del
Done.
https://codereview.appspot.com/23740043/diff/1/charmworld/views/tests/test_metrics_api.py#newcode20 charmworld/views/tests/test_metrics_api.py:20: On 2013/11/08 19:03:29, bac wrote: > Thank you for not piling onto the other test files.
Yeah, we need to start breaking those files apart. A few are way over the weight limit.
https://codereview.appspot.com/23740043/
« Back to merge proposal
I forgot to publish my draft comments on Friday. I know everyone has
been waiting patiently for them.
https:/ /codereview. appspot. com/23740043/ diff/1/ charmworld/ testing/ factory. py testing/ factory. py (right):
File charmworld/
https:/ /codereview. appspot. com/23740043/ diff/1/ charmworld/ testing/ factory. py#newcode365 testing/ factory. py:365: makeBundle = make_bundle
charmworld/
On 2013/11/08 19:03:29, bac wrote:
> You sure you'd not like to just fix them?
I'd like to... but I figured a mechanical branch later would be better.
https:/ /codereview. appspot. com/23740043/ diff/1/ charmworld/ views/api/ __init_ _.py views/api/ __init_ _.py (right):
File charmworld/
https:/ /codereview. appspot. com/23740043/ diff/1/ charmworld/ views/api/ __init_ _.py#newcode511 views/api/ __init_ _.py:511: bundle_metric_types =
charmworld/
('deployments')
Good eye. Fixed.
https:/ /codereview. appspot. com/23740043/ diff/1/ charmworld/ views/api/ __init_ _.py#newcode521 views/api/ __init_ _.py:521: if metric_name not in metric_ types:
charmworld/
self.bundle_
On 2013/11/08 19:03:29, bac wrote:
> Huh, it works anyway.
Well, I learned something new today.
https:/ /codereview. appspot. com/23740043/ diff/1/ charmworld/ views/tests/ test_metrics_ api.py views/tests/ test_metrics_ api.py (right):
File charmworld/
https:/ /codereview. appspot. com/23740043/ diff/1/ charmworld/ views/tests/ test_metrics_ api.py# newcode2 views/tests/ test_metrics_ api.py: 2:
charmworld/
On 2013/11/08 19:03:29, bac wrote:
> del
Done.
https:/ /codereview. appspot. com/23740043/ diff/1/ charmworld/ views/tests/ test_metrics_ api.py# newcode20 views/tests/ test_metrics_ api.py: 20:
charmworld/
On 2013/11/08 19:03:29, bac wrote:
> Thank you for not piling onto the other test files.
Yeah, we need to start breaking those files apart. A few are way over
the weight limit.
https:/ /codereview. appspot. com/23740043/