So I'll be making the same changes in LB as well now
monkey
I think that's reasonable. I think it would be good to show the full logo *somewhere* at least once for each project (I guess on the homepage makes sense)
the daily jams one - is that testing flag no longer needed?
If we want to run it in prod
lucifer
yes testing is always set to False in prod so I am assuming it was for tests which are not working without it as well. so not needed imo
mayhem
Thanks Lucifer!
alastairp
or for running in a dev environment so that we can quickly submit a few things?
but I guess that's DEBUG?
lucifer
the test had return_values instead of return_value which might be why it wasn't working properly without that flag.
alastairp
ah right - a few different things
mayhem
Ohhhh, the daily jam looks good!
alastairp
I see the earlier TESTING guard too
lucifer: so it looks like this completely removes the `users_to_process = ["mr_monkey", "rob", "akshaaatt", "Damselfish", "lucifer", "alastairp", "CatCat", "atj"]`
oh no, I missed like 24/25
lucifer
those are used for the other recs playlists
alastairp
like->line
mayhem: is there indoor bike parking space? (I guess there's no furniture, so...)
lucifer
i'll remove the other testing bit as well from that method and put it in a mock instead.
mayhem: to confirm those TESTING guards in the run troi function are for tests only?
mayhem
Alastairp: yes, plenty
alastairp
lucifer: fyi I think he's quite busy with guests for the rest of the night, maybe better tomorrow or next week for the important questions :)
lucifer
ah right 👍
mayhem
What Alastairp said.
:-)
alastairp
ok, on my way
BrainzGit
[sir] 14yvanzo opened pull request #135 (03master…search-452-all-url-rels): SEARCH-452, SEARCH-677: Index all URL relationships and fix indexing disambiguation comments https://github.com/metabrainz/sir/pull/135