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

Feature/register npm #1

Merged
merged 4 commits into from
Aug 29, 2017
Merged

Feature/register npm #1

merged 4 commits into from
Aug 29, 2017

Conversation

seonim-ryu
Copy link
Member

No description provided.

Copy link

@youjung-hong youjung-hong left a comment

Choose a reason for hiding this comment

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

[코드리뷰 완료] 수고하셨습니다~

"karma-sourcemap-loader": "^0.3.7",
"karma-webdriver-launcher": "git+https://github.com/everedifice/karma-webdriver-launcher.git",
"karma-webdriver-launcher": "git+https://github.com/nhnent/karma-webdriver-launcher.git",

Choose a reason for hiding this comment

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

버전명 명시해야겠네요~

"karma-webdriver-launcher": "git+https://github.com/nhnent/karma-webdriver-launcher.git#v1.1.0"

Copy link
Member Author

Choose a reason for hiding this comment

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

이거 일이군요 ;ㅅ; 감사요~

'IE10',
'IE11',
'Edge',
'Chrome-WebDriver'

Choose a reason for hiding this comment

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

여기두 웹 드라이버 설정이 빠졌네용~
그리고 IE 버전 스트링으로 감싸는 것두요~

'Safari-WebDriver': {
    base: 'WebDriver',
    config: webdriverConfig,
    browserName: 'safari'
}
'Firefox-WebDriver'
// 'Safari-WebDriver' // active only when safari test is needed

Copy link
Member Author

Choose a reason for hiding this comment

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

저번에 사파리 웹드라이브 문제있다고 해서 제외한거였는데. 이제 이슈 없으니 다시 넣어야겟네요ㅎ

"NHNent. FE Development Lab <dl_javascript@nhnent.com>"
],
"main": [
"dist/tui-project-name.js"

Choose a reason for hiding this comment

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

엇!

"dist/tui-virtual-scroll.js"

Copy link
Member Author

Choose a reason for hiding this comment

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

헙! 수정 완료!

"package.json"
],
"dependencies": {
"tui-code-snippet": "~1.2.5"

Choose a reason for hiding this comment

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

여기도 "^1.2.5"요~

Copy link
Member Author

Choose a reason for hiding this comment

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

근데 생각해보면.. bower 버전이 애매하군뇨

Copy link

@youjung-hong youjung-hong Aug 29, 2017

Choose a reason for hiding this comment

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

큽... 무슨 말인지 모르겠.. 공부를.. 해야...겠네용

"tui": true,
"loadFixtures": true
}
};

Choose a reason for hiding this comment

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

이 파일은 없어도 되겠네요~

https://eslint.org/docs/user-guide/configuring#configuration-cascading-and-hierarchy

When ESLint traverses into the tests/ directory, it will then use your-project/tests/.eslintrc in addition to your-project/.eslintrc. So your-project/tests/test.js is linted based on the combination of the two.

프로젝트 루트에 있는 .eslintrc랑 내용이 같아서 필요없어보이네요~
max-nested-callbacks처럼 test 파일들에만 적용할 룰들을 추가할 때만 필요하겠네용 ;ㅅ;

module.exports = {
    "rules": {
        "max-nested-callbacks": 0
    }
};

Copy link
Member Author

Choose a reason for hiding this comment

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

테스트하다가 파일 삭제 안된채로 올라갔나보네요 ㅠㅠ 감사요~~ 저 파일은 필요없어요ㅎ

@seonim-ryu seonim-ryu merged commit 2933d82 into develop Aug 29, 2017
@seonim-ryu seonim-ryu deleted the feature/register-npm branch August 29, 2017 01:53
@seonim-ryu seonim-ryu restored the feature/register-npm branch August 29, 2017 01:54
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.

3 participants