Skip to content

Conversation

@leaderofARS
Copy link

@leaderofARS leaderofARS commented Dec 2, 2025

Hi! I was reading through
activations.py
to better understand how the library handles activation mappings, and I noticed a small opportunity to clean up the
get_activation
function.

Since the if block returns early, the else block isn't actually needed. I've removed it to flatten the logic a bit. I know it's a tiny change, but I figured it was worth polishing up to keep the code readable.

Simplifies control flow by removing unnecessary else block after return.
@Rocketknight1
Copy link
Member

Hey! Can you run pip install -e .[quality] followed by make style to get the code quality tests to pass?

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