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

Add ability to setup repo with env #46

Merged
merged 1 commit into from Sep 10, 2016
Merged

Conversation

marocchino
Copy link
Contributor

I should use github_token since I can not use ssh key in my deployment env.
Also, I can not write in _config.yml because it is public repo.

So, I make this PR. :3

@coveralls
Copy link

coveralls commented Sep 10, 2016

Coverage Status

Coverage increased (+0.3%) to 87.64% when pulling dc393be on marocchino:env-repo into 498f637 on hexojs:master.

@Xuanwo
Copy link
Contributor

Xuanwo commented Sep 10, 2016

@coveralls
Copy link

coveralls commented Sep 10, 2016

Coverage Status

Coverage increased (+0.2%) to 88.235% when pulling 373575a on marocchino:env-repo into b4ec757 on hexojs:master.

@marocchino
Copy link
Contributor Author

marocchino commented Sep 10, 2016

Oops. sorry @Xuanwo I fixed it. :bowtie:

@Xuanwo
Copy link
Contributor

Xuanwo commented Sep 10, 2016

LGTM

@Xuanwo Xuanwo merged commit 530fad2 into hexojs:master Sep 10, 2016
@marocchino marocchino deleted the env-repo branch September 10, 2016 20:51
@JackuB
Copy link

JackuB commented Jan 4, 2017

Is this feature released on npm? Tried to use it today, but had no luck. It was merged after current 0.2.0 that's on npm.

@NoahDragon
Copy link
Member

@JackuB You could modify the package.json to set:

"hexo-deployer-git": "hexojs/hexo-deployer-git"

Then run npm install, it will get the latest development version.

@JackuB
Copy link

JackuB commented Jan 10, 2017

@NoahDragon yeah, but that kind of misses the point of versioning/package system. Switching to something like https://github.com/semantic-release/semantic-release can automate release process - I can prepare a PR with semrel if maintainer is interested.

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.

None yet

5 participants