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.

Sign up for GitHub

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

Jump to bottom

[WeeklyReport] jiangziyan-693 2023.12.10~2023.12.24 #73

Merged
merged 4 commits into from
Dec 26, 2023
Merged

[WeeklyReport] jiangziyan-693 2023.12.10~2023.12.24 #73

merged 4 commits into from
Dec 26, 2023

Conversation

jiangziyan-693
Copy link
Contributor

No description provided.

@jiangziyan-693 jiangziyan-693 mentioned this pull request Dec 22, 2023
Closed
26 tasks
@sunzhongkai588
Copy link
Contributor

突然发现，你文件名后面都缺了格式后缀 .md，前几期周报也是，都加一下吧～

SigureMo
SigureMo reviewed Dec 26, 2023
View reviewed changes
Copy link
Contributor

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

注意文件使用 .md 后缀，之前的文件里已经有没有使用 .md 后缀的了，也需要修改，我们 repo 被识别成其他语言的了

image

@ooooo-create 以后 review 的时候可以也检查下这个

@SigureMo
Copy link
Contributor

突然发现，你文件名后面都缺了格式后缀 .md，前几期周报也是，都加一下吧～

喔喔喔你已经发现了啊，那我溜了

@ooooo-create
Copy link
Collaborator

注意文件使用 .md 后缀，之前的文件里已经有没有使用 .md 后缀的了，也需要修改，我们 repo 被识别成其他语言的了

image @ooooo-create 以后 review 的时候可以也检查下这个

好的好的😂😂

SigureMo
SigureMo approved these changes Dec 26, 2023
View reviewed changes
Copy link
Contributor

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

之前的已经在 #87 修改完并 merge

@ooooo-create @sunzhongkai588 看看内容呢

@ooooo-create ooooo-create merged commit c1194d0 into PFCCLab:main Dec 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ooooo-create ooooo-create ooooo-create approved these changes

@SigureMo SigureMo SigureMo approved these changes

Assignees
No one assigned
Labels
weekly reports
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jiangziyan-693 @sunzhongkai588 @SigureMo @ooooo-create