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

feat: 20240728_CI/CD에 대해서_윤해진 #63

Merged
merged 3 commits into from
Aug 6, 2024
Merged

Conversation

haejinyun
Copy link
Contributor

No description provided.

@haejinyun haejinyun requested a review from anso33 as a code owner July 28, 2024 13:47
@soaple soaple self-requested a review July 29, 2024 06:43
Copy link

@soaple soaple left a comment

Choose a reason for hiding this comment

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

전체적으로 잘 작성하셨고, 다시 한 번 천천히 읽어보면서 글의 흐름을 조금 더 개선해보면 좋을 것 같습니다!

@@ -0,0 +1,114 @@
---
layout: post
title: CI/CD에 대해서
Copy link

Choose a reason for hiding this comment

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

제목이 글을 내용을 함축할 수 있도록 바꿔보면 좋을 것 같습니다.
예를 들면, "CI/CD의 개념과 필요성" 등이 있겠네요~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

피드백 감사합니다! 제목 수정과 함께 내용도 조금 수정해보았습니다 😄

@haejinyun haejinyun requested a review from chan99k as a code owner August 2, 2024 14:05
@haejinyun haejinyun requested a review from soaple August 2, 2024 14:10
@anso33 anso33 removed their request for review August 2, 2024 15:59
Copy link

@soaple soaple left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@chan99k chan99k left a comment

Choose a reason for hiding this comment

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

작성하느라 고생하셨습니다. 블로그 글 발행하도록 하겠습니다.

- **코드 일관성 유지**: 여러 개발자가 동시에 작업하더라도 코드베이스의 일관성을 유지할 수 있습니다.
- **통합의 부담 감소**: 작은 단위로 자주 통합함으로써 대규모 통합의 부담을 줄일 수 있습니다.

**2.2 CD(지속적 배포)**
Copy link
Contributor

@chan99k chan99k Aug 6, 2024

Choose a reason for hiding this comment

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

CD 는 Continuous Deploy (지속적 배포) 라는 의미를 가지기도 하지만, Continuous Delivery(지속적 전달) 이라는, 지속적 배포를 포함한 더 넓은 의미로도 쓰입니다. CI/CD 라고 묶어 말할 때, 일반적으로 CD는 지속적 전달을 의미합니다.

둘의 차이를 확인해 보시는 것도 재미있는 공부가 될 것 같습니다.

@chan99k chan99k merged commit 35f54d6 into main Aug 6, 2024
2 checks passed
chan99k added a commit that referenced this pull request Aug 6, 2024
Copy link
Contributor

@tonyfromundefined tonyfromundefined left a comment

Choose a reason for hiding this comment

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

전반적으로 내용이 빈약해서 조금 더 보충하는 것이 좋을거같아요. 배경을 읽어보니 "코드 스타일"에 대한 문제가 있던데, 그런 부분을 강조해보면 어떨까 싶습니다. (실제 사용 예시나 GitHub Actions의 구동 스크린샷 등을 첨부해도 좋을거같아요)

Comment on lines +97 to +98
- **ESLint 설정**: JavaScript 코드의 일관성을 유지하기 위해 ESLint를 설정하고 자동화된 검사 과정을 추가합니다.
- **Stylelint 설정**: CSS 코드의 일관성을 유지하기 위해 Stylelint를 설정하고 자동화된 검사 과정을 추가합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

ESLint, Stylelint 설정 -> Linter 설정으로 변경하고, Linter의 종류에 ESLint, Stylelint, Biome 등이 있다고 이야기해도 좋을거같아요. Linter 이외에도 코드 퀄리티 평가를 위해 Sonarqube 같은걸 사용하기도 합니다.

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.

4 participants