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

Travis ci #249

Merged
merged 9 commits into from
Feb 2, 2019
Merged

Travis ci #249

merged 9 commits into from
Feb 2, 2019

Conversation

lens0021
Copy link
Member

issue: #242

@lens0021 lens0021 self-assigned this Jan 11, 2019
@lens0021 lens0021 requested a review from simnalamburt January 11, 2019 05:49
@simnalamburt
Copy link
Member

이번 노동톤때에 리뷰할게요 죄송해요 시간이 안나네요

@lens0021
Copy link
Member Author

앗 네 그럴려고 올린거였어요

@lens0021
Copy link
Member Author

lens0021 commented Jan 25, 2019

(참고)이 PR의 체크 실패는 install-extensions.php의 린트 때문에 일어난 것이나 해당 파일은 없어질 예정이므로 무시해도 됩니다.

@simnalamburt simnalamburt force-pushed the travis-ci branch 3 times, most recently from 822403f to f07e991 Compare February 1, 2019 17:25
Copy link
Member

@simnalamburt simnalamburt left a comment

Choose a reason for hiding this comment

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

.travis.yml 안에있는 긴 셸스크립트들 읽기힘든데 주석좀 넣어주세요 특히 반복문 근처 코드들

composer.json Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
.travis.yml Outdated Show resolved Hide resolved
.travis.yml Outdated Show resolved Hide resolved
@lens0021 lens0021 force-pushed the travis-ci branch 2 times, most recently from 40f2c66 to 5a08ea2 Compare February 2, 2019 02:24
.travis.yml Outdated Show resolved Hide resolved
.travis.yml Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
@simnalamburt simnalamburt force-pushed the travis-ci branch 2 times, most recently from e1fdaea to 846b3cc Compare February 2, 2019 11:53
1.  Create '.travis.yml'
2.  Update gitignore and dockerignore files
3.  Create composer.json and composer.lock
4.  Import .phpcs.xml from https://github.com/wikimedia/mediawiki/blob/REL1_31/.phpcs.xml
5.  Update editorconfig for composer.lock and phpcs.xml
@simnalamburt simnalamburt force-pushed the travis-ci branch 2 times, most recently from 7c9dab9 to 07075b0 Compare February 2, 2019 12:17
lens0021 and others added 3 commits February 2, 2019 21:22
It is too much to test the external codes
1.  Don't lint 'secret.php'
2.  Preperation steps are 'before_script' phase not 'install' phase
3.  Reorder some steps
4.  Don't wait until 'mediawiki_parsoid' and 'mediawiki_fastcgi' are up
5.  Add comments
simnalamburt and others added 5 commits February 2, 2019 21:22
1.  Simplify multiple codes
2.  Remove 'sleep 60'
1.  Made it not to ignore LocalSettings.php
2.  Removed redundant rules
1.  Add a badge of Travis CI
2.  Mention 'composer test'
3.  Organize instructions
@simnalamburt simnalamburt merged commit 1c7ba58 into master Feb 2, 2019
@simnalamburt simnalamburt deleted the travis-ci branch February 2, 2019 12:29
@simnalamburt simnalamburt mentioned this pull request Feb 2, 2019
21 tasks
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