Skip to content

add error message if no services are defined - #35

Open
feliksik wants to merge 3 commits into
gogo:masterfrom
feliksik:master
Open

add error message if no services are defined#35
feliksik wants to merge 3 commits into
gogo:masterfrom
feliksik:master

Conversation

@feliksik

Copy link
Copy Markdown

addresses #34

Note I didn't even test this. Feel free not to merge it. Maybe I'd have time for a more serious contribution later.

@awalterschulze

Copy link
Copy Markdown
Member

I would appreciate it if you could at least test it locally and apply a gofmt.

@feliksik

Copy link
Copy Markdown
Author

Yes sure. I cannot test it now, really. Feel free to delete, ignore, stash or teleport this PR. I really hope to be able to be more helpful, at another time.

@awalterschulze

Copy link
Copy Markdown
Member

Sorry I didn't get a notification that you updated the commit.
Are we still happy with this change?

@feliksik

Copy link
Copy Markdown
Author

I am currently no longer working with letmegrpc, I'm in Java land now... Sorry

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