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 gocui package, use jesseduffield's gocui #79

Merged
merged 1 commit into from Dec 14, 2018
Merged

Fix gocui package, use jesseduffield's gocui #79

merged 1 commit into from Dec 14, 2018

Conversation

skanehira
Copy link
Contributor

Fix import package.

@pankona
Copy link
Owner

pankona commented Dec 14, 2018

@skanehira
Thank you for creating pull request! I'll check.

@pankona
Copy link
Owner

pankona commented Dec 14, 2018

This patch makes hashira-cui go gettable but build with dependency management (with go.mod or Gopkg.toml) seems to be broken. I'll investigate how to make hashira go gettable and also build with specified dependencies.

@pankona
Copy link
Owner

pankona commented Dec 14, 2018

@skanehira
So, just a moment to merge this PR! Thanks!

@pankona
Copy link
Owner

pankona commented Dec 14, 2018

I found following modification fixes build:

  • update go.mod (and go.sum) to use jesseduffield's gocui v0.3.1
    • and replace it with pankona's gocui
  • update Gopkg.toml to refer jesseduffield's gocui

After merging this patch, I'll do additional modification on it.

@pankona
Copy link
Owner

pankona commented Dec 14, 2018

By the way, I didn't allow to run circle ci for pull request from forked repository (since it is default setting), but currently it has been enabled.

@pankona
Copy link
Owner

pankona commented Dec 14, 2018

merge!

@pankona pankona merged commit dbf6302 into pankona:master Dec 14, 2018
@pankona
Copy link
Owner

pankona commented Dec 14, 2018

@skanehira
Thank you so much your contribution!!

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

2 participants