Skip to content

Conversation

@msampathkumar
Copy link
Member

Description

Copyright header - fixing the spell check. No code changes.

Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.

Checklist

  • Please merge this PR for me once it is approved

Copyright header - fix spell check
@msampathkumar msampathkumar requested review from a team as code owners January 29, 2025 13:55
@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Jan 29, 2025
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Hi @msampathkumar and reviewers,

I'm currently reviewing this pull request and will provide my feedback shortly. In the meantime, here's a summary to help everyone get up to speed:

This pull request addresses a simple spell check issue in the copyright headers of several files within the gemma2 directory. The intent is to correct a typo in the license header.

Changes:

  • The word "WITHcontent" was changed to "WITHOUT" in the copyright headers of the following files:
    • gemma2/gemma2_predict_gpu.py (line 11)
    • gemma2/gemma2_predict_tpu.py (line 11)
    • gemma2/gemma2_test.py (line 11)

No functional code changes were made; only the copyright headers were updated.

Activity:

The pull request description indicates that the author has completed the checklist and is requesting a merge once approved.


A tiny poem to brighten your day:

A header's small fix,
A typo now corrected,
Code shines anew.

Let me know if you have any questions. I'll post my full review shortly!

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

This pull request corrects a spelling error in the copyright header. This is a purely documentation change and does not affect any code functionality. Since this is a documentation change, no style guide is applicable.

@glasnt glasnt merged commit 94779b4 into main Jan 29, 2025
14 checks passed
@glasnt glasnt deleted the sampath_fix_spellchecks branch January 29, 2025 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants