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

Fix misleading config param github_url/base_url #29

Merged
merged 5 commits into from
Jun 18, 2019

Conversation

everpcpc
Copy link
Contributor

@everpcpc everpcpc commented Apr 22, 2019

The mismatch of github_url and base_url acturally breaks github enterprise support for those actions which do not use _request.

related issue: #20

The mismatch of `github_url` and `base_url` acturally
breaks github enterprise support.
@LindsayHill
Copy link
Contributor

Sorry this one got missed earlier.

In general I agree with cleaning this up. Could do please do this:

1/ Check README.md, see if that needs any updates to reflect this change
2/ Bump the version string in pack.yaml - I suggest going to 1.0.0
3/ Add an entry in CHANGES.md

Then we can merge it

@LindsayHill LindsayHill merged commit 1765a7d into StackStorm-Exchange:master Jun 18, 2019
@LindsayHill
Copy link
Contributor

Thank you!

@everpcpc everpcpc deleted the fix branch June 18, 2019 03:14
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