-
Notifications
You must be signed in to change notification settings - Fork 16
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
initialise diff chapter #49
Conversation
6b9a58d
to
1d57642
Compare
1d57642
to
6e118c9
Compare
I'll use this PR to share some progress update as well. I've completed end-to-end implementations, associated with the following increments (some of following bullet-points correspond to a single increment):
Naturally flowing next steps:
|
1541c01
to
96e9963
Compare
I realize this is in progress, but may I merge what you have so that I have something when I start rearranging order? |
My apologies, @gvwilson - it appears that I've misunderstood the purpose of the chapter sub-directories and the associated PR increments. The initialisation of the sub-directory was in response to this comment from the associated issue:
But I wasn't sure at what frequencies should we have expected the PR increments to be merged into the repository here. I confirm I've been writing the draft of my chapter (and complete code examples) in parallel (i.e., whilst waiting for this initialisation PR to be merged). Should I seek to update this PR with my draft work, or should that be part of a separate PR? I confirm, either way, I'll be able to provide a much better idea about the content of the chapter this weekend. Basically (and it seems - entirely caused by my misunderstanding), I was waiting for this PR to be merged in first, so that I could subsequently raise other PRs corresponding to draft version increments. |
Hi @hristog - please don't apologize, we're making this up as we go along :-) I have one more week until I start a new job, and would like to tidy up what we have before then, so I've been firing off requests for merge more hastily than I should have. You've done everything right; I'll go ahead and merge this as it is and then we can see where we are. Apologies on my part for being confusing. |
Understood, @gvwilson! Thank you once again for confirming! |
Summary:
diff
chapter.