Skip to content
This repository has been archived by the owner on Mar 22, 2023. It is now read-only.

Deploy with github actions #810

Merged
merged 4 commits into from
Feb 23, 2019
Merged

Deploy with github actions #810

merged 4 commits into from
Feb 23, 2019

Conversation

marocchino
Copy link
Contributor

@marocchino marocchino commented Feb 23, 2019

fix #606

@marocchino marocchino force-pushed the new-build branch 2 times, most recently from b8ad4fa to ee1b220 Compare February 23, 2019 01:59
@marocchino marocchino changed the title ci: Deploy with github actions Deploy with github actions Feb 23, 2019
@marocchino marocchino self-assigned this Feb 23, 2019
@marocchino marocchino force-pushed the new-build branch 2 times, most recently from 6f399ee to 5f95f47 Compare February 23, 2019 02:38
@marocchino marocchino force-pushed the new-build branch 5 times, most recently from 69f4d16 to 88723f3 Compare February 23, 2019 04:01
@marocchino
Copy link
Contributor Author

marocchino commented Feb 23, 2019

88723f3 에서 성공했을때의 로그입니다 이후에 커밋정리하고 필터 적용만 했어요
https://github.com/nodejs/nodejs-ko/actions/workflow-runs/MDEwOkNoZWNrU3VpdGU2ODYzMjQ2OA==

@@ -93,6 +93,7 @@ deploy:
branch: gh-pages
name: deployer
email: [email protected]
repo: [email protected]:nodejs/nodejs-ko.git
Copy link
Contributor Author

Choose a reason for hiding this comment

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

원래는 http를 환경변수에서 꺼내 썻었습니다. rsa를 사용할 수 있게되서 설정에 적을 수 있게되었습니다.

"deploy": "hexo generate && hexo deploy",
"generate": "hexo generate",
"deploy": "hexo deploy",
"publish": "cp -r .ssh /root/. && hexo deploy",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

다음 엑션에 전달해줄수 있는 파일은 지금 경로 아래에 있는 파일만 가능한가 봅니다.
그래서 일단 지금 위치에 만들고 나중에 ~로 이동하도록 해두었습니다.

@marocchino marocchino requested a review from yous February 23, 2019 04:13
@marocchino
Copy link
Contributor Author

@outsideris 좀 첨언하면.. 이건 그냥 디플로이 안되는 거 고친거구요. 최종적으로 #793 이 되는게 좋다고 생각합니다.

Copy link
Contributor

@yous yous left a comment

Choose a reason for hiding this comment

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

👍

@outsideris outsideris merged commit 7ff4fd6 into master Feb 23, 2019
@outsideris outsideris mentioned this pull request Feb 23, 2019
@yous yous deleted the new-build branch February 23, 2019 09:17
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 this pull request may close these issues.

build setting
3 participants