Remove some duplicate code

[tremby]

This line of code was here twice, along with its comment; likely a copy and paste error at some point.

[mattrobenolt]

Ah, thanks. 🍰 🍦

[benvinegar]

Whoa whoa whoa @mattrobenolt! Talk about haste! There are plenty of good reasons why this code is duplicated!!!!

[benvinegar]

jk 👍

[tremby]

While you guys are here, what I was trying to investigate when I noticed this duplicate code is how exactly "release" is handled. Some Sentry docs suggest using a tag "version", but then Raven JS has this config option to pass "release", separate from tags, and it's rendered as if it were a tag in Sentry as far as I can tell. Is "release" special in some way? Can you point me at code or docs?

[mattrobenolt]

Yeah, we handle release as a first class citizen in the API, but internally it gets transformed into a tag. It's more like a "special" tag, since we use it for internal purposes to manage releases and whatnot.

[mattrobenolt]

Also, those docs are misleading. We should update that. A tag "version" is not the same thing as a release. It's likely these docs were written before releases were a native thing.

[tremby]

Okay, all understood. I'll switch my code to send "release" instead. Found how to send it properly in PHP Raven, too. Thanks!