Code review comment for lp://qastaging/~zhangew401/scope-aggregator/fix-lp-1524597

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

I think it would be even more clear if the key name is changed (in source and README.md) to "keyword_scope_ids".

Please make these two changes and go ahead and merge.

Thanks!

review: Approve

« Back to merge proposal