Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Return the right number of artists, not one short, lol. #2839

Merged
merged 3 commits into from
Apr 16, 2024

Conversation

mayhem
Copy link
Member

@mayhem mayhem commented Apr 16, 2024

The endpoint was returning one fewer artist than expected. I slaved for HOURS over this massive PR!!

Copy link
Member

@MonkeyDo MonkeyDo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah ! Many a developer from the four corners of the globe have slaved for hours, days, months taking apart their code piece by piece, testing each in isolation until they were satisfied that no, the problem couldn't possibly come from that piece, living every day in astonishment, fear and disgust at their own incapacity to tame the dragon they themselves brought forth form the deepest recesses of their mind (or StackOverflow), losing all sense of the passage time, only to realize much, much too late that a simple sign, a single character was to blame for the fault in the logic of their creation.
If only life's own problems could be solved so simply, after much agonizing and hair pulling, by the addition of a single equality symbol...
All things balanced, as they should be.
All in all, as a metaphor both of our own failures and our capacity to overcome adversity, a simple +1 / -1
Change thyself from within, one grain at a time.
image

@mayhem mayhem merged commit f39c156 into master Apr 16, 2024
2 checks passed
@mayhem mayhem deleted the lb-artist-radio-fixes branch April 16, 2024 10:51
MonkeyDo added a commit that referenced this pull request Apr 26, 2024
Following from the partial fix added in #2839
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants