Skip to content

[csv/zh-cn] Add translation for csv tutorials (Fixes #5291) #5292

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 5 commits into from

Conversation

AlightSoulmate
Copy link

  • I solemnly swear that this is all original content of which I am the original author
  • Pull request title is prepended with [language/lang-code] (example [python/fr] for Python in French or [java] for multiple Java translations)
  • Pull request touches only one file (or a set of logically related files with similar changes made)
  • Content changes are aimed at intermediate to experienced programmers (this is a poor format for explaining fundamental programming concepts)
  • If you've changed any part of the YAML Frontmatter, make sure it is formatted according to CONTRIBUTING.md

@AlightSoulmate AlightSoulmate changed the title [csv/zh-cn] Add translation for csv tutorials #5291 [csv/zh-cn] Add translation for csv tutorials Fixes #5291 Apr 1, 2025
@AlightSoulmate AlightSoulmate changed the title [csv/zh-cn] Add translation for csv tutorials Fixes #5291 [csv/zh-cn] Add translation for csv tutorials (Fixes #5291) Apr 1, 2025
@nbehrnd
Copy link
Contributor

nbehrnd commented Apr 1, 2025

@AlightSoulmate Note the style guidelines in CONTRIBUTING.md recommend 80 characters per line as maximum. While it is not always possible to comply to this rule in blocks of source code, or link addresses, it usually is for running text.

@AlightSoulmate
Copy link
Author

@AlightSoulmate Note the style guidelines in CONTRIBUTING.md recommend 80 characters per line as maximum. While it is not always possible to comply to this rule in blocks of source code, or link addresses, it usually is for running text.

I've optimized two sections for brevity.

@imba-tjd
Copy link
Collaborator

imba-tjd commented Apr 1, 2025

unfortunately there was a relatively huge change on en version of csv: #5286
If this got merged, it would be outdated in no time.

@AlightSoulmate
Copy link
Author

AlightSoulmate commented Apr 1, 2025

@imba-tjd Oops, I have not noticed that, I have closed this commit and will try another one in the future

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.

3 participants