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

INTERNAL: Remove whitespace in etc files #352

Merged
merged 1 commit into from
Jan 24, 2025
Merged

Conversation

ing-eoking
Copy link
Collaborator

🔗 Related Issue

⌨️ What I did

  • 기타 파일에서 whitespace를 제거합니다.

Copy link
Contributor

@jhpark816 jhpark816 left a comment

Choose a reason for hiding this comment

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

리뷰 완료

config/pandora-plugin.am \
config/pandora-plugin.ac \
config/pandora-plugin \
config/pandora-plugin.ini
Copy link
Contributor

Choose a reason for hiding this comment

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

tabstop=2 설정 경우로 맞추면 될 것 같습니다.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

수정되었습니다.

ChangeLog Outdated
* [FIX] if MEMCACHED_ERRNO entered in memcached_set_error(),
return instead of assert.
* [FEATURE] enable zookeeper multi thread mode.
* [FIX] if MEMCACHED_ERRNO entered in memcached_set_error(),
Copy link
Contributor

Choose a reason for hiding this comment

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

여기도 tabstop=2 인 경우에 보이는 형태로 맞추면 좋겠습니다.

@ing-eoking ing-eoking marked this pull request as draft January 24, 2025 05:25
@ing-eoking ing-eoking marked this pull request as ready for review January 24, 2025 05:33
@jhpark816 jhpark816 merged commit f56842d into naver:develop Jan 24, 2025
1 check passed
@ing-eoking ing-eoking deleted the clean6 branch January 24, 2025 05:59
@ing-eoking
Copy link
Collaborator Author

PR 수정 과정에서 commit 메시지가 잘못 들어가서 rebase전까지 잠시 merge는 중단부탁드립니다.
완료되면 알려드리겠습니다.

@ing-eoking
Copy link
Collaborator Author

@jhpark816

완료되었습니다.

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.

2 participants