Skip to content
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

fix: Add MustPutInputFile function #144

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Conversation

qianxi66
Copy link
Contributor

@qianxi66 qianxi66 commented Nov 6, 2024

检测上传的inputfile换行符格式是否符合要求。

  1. 将所有 '\r\n' 替换为 '\n'
  2. 如果文件结尾的最后一个字符不是 '\n', 加一个
  3. 通过增加sanitize变量作为开关控制这个行为

Copy link

github-actions bot commented Nov 6, 2024

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


1 out of 2 committers have signed the CLA.
@qianxi66
@Qianxi
qianxi seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request

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.

1 participant