Code review comment for lp:~reedobrien/charmworld/ngrams

Revision history for this message
Richard Harding (rharding) wrote :

QA is mixed. The results seem no worse than current so let's try this out. I did try out search for 'couch' which I would have expected to ngram match for 'couchdb' but it did not. I also tried 'build' which found 'buildbot'. I'm not sure if we hit on other matching criteria though.

As for review, let's take advantage of the ability to skip a test. I think nose will respect @unittest.skip("") rather than comment out a large block of code.

review: Needs Fixing

« Back to merge proposal