Skip to content

Need help with goland support#67

Merged
ashald merged 1 commit intoashald:developfrom
Kelnor277:GolandSupport
Oct 7, 2018
Merged

Need help with goland support#67
ashald merged 1 commit intoashald:developfrom
Kelnor277:GolandSupport

Conversation

@Kelnor277
Copy link
Copy Markdown
Contributor

I think I this should work? Unless there's some special cases for go run configs. However, I can't seem to figure out how to build it, in order to test. Any help in how to build the plugin for local testing would be appreciated.

Thanks.

@Kelnor277
Copy link
Copy Markdown
Contributor Author

Just saw the migrate to gradle PR. That might help me.

@ashald
Copy link
Copy Markdown
Owner

ashald commented Oct 7, 2018

Thanks for the contribution! I can confirm that it works - great job. Given that #68 was merged I will go ahead and merge this PR so I can move forward with the release.

@ashald ashald closed this Oct 7, 2018
@ashald ashald reopened this Oct 7, 2018
@ashald ashald merged commit d0615a5 into ashald:develop Oct 7, 2018
@Kelnor277
Copy link
Copy Markdown
Contributor Author

Nice, must be the first time I ever wrote code that worked on the first try... and by wrote I mean copied your other code and changed the class name to the go run config.

@ashald
Copy link
Copy Markdown
Owner

ashald commented Oct 7, 2018

There were few mismatches in configuration [but it's always super tricky to figure them out for IDEA - I struggle with it all the time], but code was fine. 👍
And I do love copy-pasting existing code and adjusting it as needed as well. 😉

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