Skip to content

just a lil spelling fix in the demo notebook#1554

Open
Ruler10123 wants to merge 1 commit into
google:mainfrom
Ruler10123:spelling-fix
Open

just a lil spelling fix in the demo notebook#1554
Ruler10123 wants to merge 1 commit into
google:mainfrom
Ruler10123:spelling-fix

Conversation

@Ruler10123

Copy link
Copy Markdown

No description provided.

@R-behera R-behera left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks for cleaning up the spelling in the notebook. One thing that stood out in the diff is that the change also re-saves a fair amount of notebook metadata and cell ordering noise beyond the typo itself. If you still have the branch handy, could you try to keep the diff scoped to the text fix only? That would make future notebook reviews much easier and reduce the risk of unrelated metadata churn slipping in with content edits.

@Ruler10123

Copy link
Copy Markdown
Author

Thanks for the notification! Sorry if this is a weird question, but how did the metadata churn/change randomly? All I did was go in and change one line.

Best,
Sean

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