executorch icon indicating copy to clipboard operation
executorch copied to clipboard

Propose fix some typos

Open jeis4wpi opened this issue 1 month ago • 3 comments

Summary

Propose fix some typos.

jeis4wpi avatar Nov 11 '25 20:11 jeis4wpi

:link: Helpful Links

:test_tube: See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/15763

Note: Links to docs will display an error until the docs builds have been completed.

:heavy_exclamation_mark: 2 Active SEVs

There are 2 currently active SEVs. If your PR is affected, please view them below:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

pytorch-bot[bot] avatar Nov 11 '25 20:11 pytorch-bot[bot]

@pytorchbot label "release notes: none"

jeis4wpi avatar Nov 11 '25 21:11 jeis4wpi

@jeis4wpi Thank you for catching and fixing these typos!

manuelcandales avatar Nov 14 '25 21:11 manuelcandales

@jeis4wpi Please fix the lint errors

https://github.com/pytorch/executorch/blob/main/CONTRIBUTING.md#lintrunner

mergennachin avatar Nov 17 '25 16:11 mergennachin

lintrunner
  FLAKE8 success!
  UFMT success!
  CLANGFORMAT success!
  CMAKEFORMAT success!
  ETCAPITAL success!
  NEWLINE success!
  NOSTDINC success!
  MYPY success!
  TORCH_AO_IMPORT success!
  ETVKNODEBUG success!
  CMAKE success!
  NOTORCHINC success!
  LICENSELINT success!
ok No lint issues.

Are the lint issues now resolved?

jeis4wpi avatar Nov 17 '25 17:11 jeis4wpi

It seems like this PR includes commits from other authors, maybe rebase doesn't work properly, can you double check?

cccclai avatar Nov 25 '25 16:11 cccclai

Yes, I tried to rebase today. Not sure what I did wrong.

jeis4wpi avatar Nov 25 '25 17:11 jeis4wpi

Looks like I should close this PR with changes included that I didn't make.

jeis4wpi avatar Nov 25 '25 20:11 jeis4wpi