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

Patch add gitlab support #133

Closed
wants to merge 9 commits into from

Conversation

AtarisMio
Copy link

Add gitlab support and refactor code.

@evocateur evocateur requested a review from Turbo87 May 14, 2019 21:34
@AtarisMio
Copy link
Author

#23

@AtarisMio
Copy link
Author

#3

@Turbo87
Copy link
Contributor

Turbo87 commented May 19, 2019

@AtrisMio hey 👋 thanks for working on this! I'm currently on vacation and will review this properly once I get back.

@AtarisMio
Copy link
Author

@Turbo87 thx for that.

@my9074
Copy link

my9074 commented May 31, 2019

@AtrisMio Hey~ It's good ~~ Does this PR support private gitlab

@AtarisMio
Copy link
Author

@my9074 Yes, It can use environment of GITLAB_SERVER.

GITLAB_SERVER=https://gitlab.my-private-site.com GITLAB_AUTH=xxxxx lerna-changelog

@Turbo87
Copy link
Contributor

Turbo87 commented Jun 22, 2019

@AtrisMio this seems to be a pretty large refactoring. I'm not sure how to properly review this. Can you break this down into smaller chunks and explain them separately? 🤔

@AtarisMio
Copy link
Author

@Turbo87 thx for reply, I will break this down in the near future.

@AtarisMio AtarisMio closed this Jun 26, 2019
@koka0012
Copy link

Any update?

this.ignoreCommitters = config.ignoreCommitters;
this.auth = this.getAuthToken();
if (!this.auth) {
throw new ConfigurationError("Must provide GITHUB_AUTH");

Choose a reason for hiding this comment

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

or GITLAB_AUTH if --git-provider=gitlab ?

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