Skip to content

update : add new team's info #1669

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

Merged
merged 2 commits into from
Jul 14, 2025
Merged

update : add new team's info #1669

merged 2 commits into from
Jul 14, 2025

Conversation

SamTheKorean
Copy link
Contributor

변경사항

5기 기수 정보를 추가하였습니다!

@SamTheKorean
Copy link
Contributor Author

Readme 수정인데 파일명 검사가 패스가 안되는데 룰 변경이 필요할 것 같습니다!

Copy link
Member

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

프로젝트 링크만 고쳐주시면 될 것 같습니다. CI 실패하는 문제는 maintenance 태그 추가하여 해결해드렸습니다.

README.md Outdated
@@ -13,6 +13,7 @@

## 차수

- 5기 (Jul 20, 2025 - Nov 1, 2025): [프로젝트](https://github.com/orgs/DaleStudy/projects/16), [팀](https://github.com/orgs/DaleStudy/teams/leetcode05)
Copy link
Member

Choose a reason for hiding this comment

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

프로젝트의 뷰를 기존 기수와 일관되게 하면 좋을 것 같습니다. 기본 뷰는 기수가 종료되면 아무 PR도 나오지 않거든요.

Suggested change
- 5기 (Jul 20, 2025 - Nov 1, 2025): [프로젝트](https://github.com/orgs/DaleStudy/projects/16), [](https://github.com/orgs/DaleStudy/teams/leetcode05)
- 5기 (Jul 20, 2025 - Nov 1, 2025): [프로젝트](https://github.com/orgs/DaleStudy/projects/16/views/3), [](https://github.com/orgs/DaleStudy/teams/leetcode05)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

View에 대한 이해가 없었군요! 피드백 감사합니다!

@SamTheKorean SamTheKorean merged commit d1d277e into main Jul 14, 2025
1 check passed
@SamTheKorean SamTheKorean deleted the github branch July 14, 2025 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants