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 coding.net token support #168

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

zhangnew
Copy link

coding.net needs token name and token to authenticate.
see https://help.coding.net/docs/project/features/deploy-tokens.html

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.1%) to 89.437% when pulling f554e3a on zhangnew:master into a4d02c9 on hexojs:master.

@coveralls
Copy link

coveralls commented Mar 14, 2020

Coverage Status

Coverage increased (+0.3%) to 90.845% when pulling d913a40 on zhangnew:master into a4d02c9 on hexojs:master.

lib/parse_config.js Outdated Show resolved Hide resolved
README.md Outdated
@@ -66,6 +68,7 @@ deploy:
- Defaults to `coding-pages` on Coding.net.
- Otherwise defaults to `master`.
- **token**: Optional token value to authenticate with the repo. Prefix with `$` to read token from environment variable (recommended). Repo must be a http(s) url. [More details](#deploy-with-token).
- **token_name** coding.net needs token name and token to authenticate. [More details](#deploy-with-token)
Copy link
Member

Choose a reason for hiding this comment

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

Should mentioned that the configuration only valid for coding.net.

Copy link
Author

Choose a reason for hiding this comment

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

fixed

Co-Authored-By: Sukka <isukkaw@gmail.com>
@zhangnew
Copy link
Author

Can this commit be merged ?

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

3 participants