Skip to content
This repository has been archived by the owner on Nov 29, 2022. It is now read-only.

ridibooks/php-core 의존성 제거 #14

Closed
namenu opened this issue Feb 13, 2018 · 1 comment · Fixed by #29
Closed

ridibooks/php-core 의존성 제거 #14

namenu opened this issue Feb 13, 2018 · 1 comment · Fixed by #29
Assignees

Comments

@namenu
Copy link
Contributor

namenu commented Feb 13, 2018

본 패키지가 플랫폼팀 모듈에 전반에 사용되고 있어 php-core의 버전 호환이 문제가 되고 있습니다.

  1. MsgException
  2. GnfConnectionProvider
  3. SentryHelper

위 세 가지 기능을 위해 php-core에 의존하고 있으나, 1&2는 플랫폼팀에서만 사용되므로 단순히 옮기면 됩니다.
3은 구현을 그대로 가져오는 것이 전체를 참조하는 것 보다는 이득인 상황입니다.

@ssaifriend
Copy link
Contributor

1,3은 그대로 옮기는게 좋은 방향일 것 같습니다.
그런데, 2번 GnfConnectionProvider에서 ConnectionProvider를 참조하고 있는데,
ConnectionProvider 참조를 걷어내고 내부 코드 일부를 GnfConnectionProvider로 옮기는게 좋을 것 같습니다.

@ssaifriend ssaifriend self-assigned this Feb 26, 2018
@ssaifriend ssaifriend assigned Goliathtom and unassigned ssaifriend May 21, 2019
@ssaifriend ssaifriend linked a pull request Apr 22, 2020 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants