Skip to content

Fix misc pylint issues.#3237

Merged
copybara-service[bot] merged 1 commit intomainfrom
nicogrande/fix-pylint-new
Feb 25, 2026
Merged

Fix misc pylint issues.#3237
copybara-service[bot] merged 1 commit intomainfrom
nicogrande/fix-pylint-new

Conversation

@NicoGrande
Copy link
Copy Markdown
Collaborator

Description

Fix misc pylint issues.

Tests

CI

Checklist

Before submitting this PR, please make sure (put X in square brackets):

  • I have performed a self-review of my code. For an optional AI review, add the gemini-review label.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have run end-to-end tests tests and provided workload links above if applicable.
  • I have made or will make corresponding changes to the doc if needed, including adding new documentation pages to the relevant Table of Contents (toctree directive) as explained in our documentation.

Comment thread src/maxtext/layers/decoders.py Outdated
@NicoGrande NicoGrande force-pushed the nicogrande/fix-pylint-new branch from 04f1549 to bad4fc3 Compare February 24, 2026 23:01
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 24, 2026

Codecov Report

❌ Patch coverage is 37.50000% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/maxtext/layers/decoders.py 37.50% 1 Missing and 4 partials ⚠️

📢 Thoughts on this report? Let us know!

@copybara-service copybara-service Bot merged commit 8bb73fc into main Feb 25, 2026
51 of 54 checks passed
@copybara-service copybara-service Bot deleted the nicogrande/fix-pylint-new branch February 25, 2026 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants