You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I don't know how the long code snippet will read in a book format, as it might stretch into multiple pages. I had a lengthy code section in the android chapter (https://github.com/xrd/advanced-github-oreilly/blob/master/chapter-07-android.asciidoc#code-to-talk-to-github). I ended up breaking the large main function into a bunch of small subroutines, discussing the high level main function, and then having a different smaller section for each of the necessary subroutines. Something like this abbreviated main routine with the functions like main, webhookHandler and the others functions tubbed out. Then you could discuss the strategy after displaying this code snippet and afterwards have a level 3 header which discusses the implementation (and show the code sample) for each function. If you want me to take a stab at refactoring this, I'd be happy to.
Section: https://github.com/xrd/advanced-github-oreilly/blob/jpoz/chapter-10-go.asciidoc#setting-up-our-webhooks-endpoint
I don't know how the long code snippet will read in a book format, as it might stretch into multiple pages. I had a lengthy code section in the android chapter (https://github.com/xrd/advanced-github-oreilly/blob/master/chapter-07-android.asciidoc#code-to-talk-to-github). I ended up breaking the large main function into a bunch of small subroutines, discussing the high level main function, and then having a different smaller section for each of the necessary subroutines. Something like this abbreviated main routine with the functions like
main
,webhookHandler
and the others functions tubbed out. Then you could discuss the strategy after displaying this code snippet and afterwards have a level 3 header which discusses the implementation (and show the code sample) for each function. If you want me to take a stab at refactoring this, I'd be happy to.The text was updated successfully, but these errors were encountered: