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

NPM에 올려주세요(2) #58

Merged
merged 4 commits into from Apr 21, 2016

Conversation

Projects
None yet
3 participants
@item4
Contributor

item4 commented Dec 14, 2015

#51 에서 PR을 올리면 NPM에 올라갈 가능성이 올라간다고 하셨어서 만들어 왔습니다. 결코 JS 패키징으로 @dahlia 님을 괴롭히려는게 아닙니다.

  • fonts/ 내의 내용은 ln 사용
  • less 지원
  • scss 지원
  • webpack 등의 환경에서 쓰일 경우를 대비해서 @spoqa-han-sans-path/$spoqa-han-sans-path 변수 지원

머지 후 해야할 일

  1. (권장) package.jsoncontributors속성 작성.
  2. (필수) publish -- https://docs.npmjs.com/getting-started/publishing-npm-packages // 대행 불가능한 작업
  3. (필수) 설치 후 사용 가능한지 확인
@dahlia

This comment has been minimized.

Contributor

dahlia commented Apr 21, 2016

뒤늦게 리뷰를 드려서 죄송합니다.

  • fonts/ 내의 내용은 ln 사용

제 기억이 맞자면 Git 수준에서 Unix hard links는 preserve하지 않는다고 알고 있습니다. 중복된 파일이 2벌씩 생기게 된 것 아닌가요?

@item4

This comment has been minimized.

Contributor

item4 commented Apr 21, 2016

@dahlia 중복된 파일이 두 벌 생긴것이 맞아서 기존 파일을 그대로 사용하도록 변경하였습니다

@@ -0,0 +1,29 @@
{
"name": "spoqa-han-sans",
"version": "1.0.0",

This comment has been minimized.

@dahlia

dahlia Apr 21, 2016

Contributor

최근에 릴리스된 버전은 1.0.1입니다.

This comment has been minimized.

@dahlia

dahlia Apr 21, 2016

Contributor

왜 1.0.0으로 적으셨는지 몰랐는데, 공식 README.md가 여전히 1.0.0이라고 버전을 표기하고 있군요. README.md의 버전 표기도 최근 버전으로 올려둬야겠네요…

@dahlia dahlia merged commit d2d6a42 into spoqa:master Apr 21, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@dahlia

This comment has been minimized.

Contributor

dahlia commented Apr 21, 2016

기여해주셔서 감사합니다. NPM 저장소에는 제가 올려보도록 하겠습니다.

@dahlia

This comment has been minimized.

Contributor

dahlia commented Apr 21, 2016

@item4 덕분에 NPM 저장소에 잘 올라갔습니다. npm version

@lqez

This comment has been minimized.

lqez commented Apr 22, 2016

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment