From c7d9e100516d00ec91ba99d5285d1b0fc9a70444 Mon Sep 17 00:00:00 2001
From: Peter Hunt <peter.hunt@bglgroup.co.uk>
Date: Wed, 10 Jul 2019 09:23:44 +0100
Subject: [PATCH 01/13] :arrow_up: Make templates compatible with Node 12

---
 generate/templates/manual/clone/clone.cc      | 24 ++++++-------
 .../manual/commit/extract_signature.cc        | 14 ++++----
 generate/templates/manual/filter_list/load.cc | 36 +++++++++----------
 .../templates/manual/filter_source/repo.cc    |  6 ++--
 .../manual/patches/convenient_patches.cc      |  8 ++---
 generate/templates/manual/remote/ls.cc        |  2 +-
 .../manual/repository/get_references.cc       |  4 +--
 .../manual/repository/get_remotes.cc          |  4 +--
 .../manual/repository/get_submodules.cc       |  4 +--
 .../manual/repository/refresh_references.cc   | 12 +++----
 .../templates/manual/revwalk/commit_walk.cc   |  4 +--
 .../templates/manual/revwalk/fast_walk.cc     | 12 +++----
 .../manual/revwalk/file_history_walk.cc       | 12 +++----
 .../templates/manual/src/filter_registry.cc   | 24 ++++++-------
 .../templates/manual/src/git_buf_converter.cc |  2 +-
 .../templates/manual/src/nodegit_wrapper.cc   |  2 +-
 .../manual/src/promise_completion.cc          | 14 ++++----
 .../manual/src/str_array_converter.cc         |  2 +-
 generate/templates/partials/async_function.cc | 14 ++++----
 .../templates/partials/callback_helpers.cc    |  4 +--
 .../templates/partials/convert_from_v8.cc     | 14 ++++----
 generate/templates/partials/convert_to_v8.cc  |  8 ++---
 .../templates/partials/field_accessors.cc     | 10 +++---
 generate/templates/partials/sync_function.cc  |  2 +-
 .../templates/templates/struct_content.cc     |  4 +--
 25 files changed, 121 insertions(+), 121 deletions(-)

diff --git a/generate/templates/manual/clone/clone.cc b/generate/templates/manual/clone/clone.cc
index 144be221d..2cab928bf 100644
--- a/generate/templates/manual/clone/clone.cc
+++ b/generate/templates/manual/clone/clone.cc
@@ -33,7 +33,7 @@ NAN_METHOD(GitClone::Clone) {
   // start convert_from_v8 block
   const char *from_url = NULL;
 
-  String::Utf8Value url(info[0]->ToString());
+ Nan::Utf8String url(Nan::To<v8::String>(info[0]).ToLocalChecked());
   // malloc with one extra byte so we can add the terminating null character
   // C-strings expect:
   from_url = (const char *)malloc(url.length() + 1);
@@ -50,7 +50,7 @@ NAN_METHOD(GitClone::Clone) {
   // start convert_from_v8 block
   const char *from_local_path = NULL;
 
-  String::Utf8Value local_path(info[1]->ToString());
+ Nan::Utf8String local_path(Nan::To<v8::String>(info[1]).ToLocalChecked());
   // malloc with one extra byte so we can add the terminating null character
   // C-strings expect:
   from_local_path = (const char *)malloc(local_path.length() + 1);
@@ -67,7 +67,7 @@ NAN_METHOD(GitClone::Clone) {
   // start convert_from_v8 block
   const git_clone_options *from_options = NULL;
   if (info[2]->IsObject()) {
-    from_options = Nan::ObjectWrap::Unwrap<GitCloneOptions>(info[2]->ToObject())
+    from_options = Nan::ObjectWrap::Unwrap<GitCloneOptions>(Nan::To<v8::Object>(info[2]).ToLocalChecked())
                        ->GetValue();
   } else {
     from_options = 0;
@@ -80,11 +80,11 @@ NAN_METHOD(GitClone::Clone) {
   CloneWorker *worker = new CloneWorker(baton, callback);
 
   if (!info[0]->IsUndefined() && !info[0]->IsNull())
-    worker->SaveToPersistent("url", info[0]->ToObject());
+    worker->SaveToPersistent("url", Nan::To<v8::Object>(info[0]).ToLocalChecked());
   if (!info[1]->IsUndefined() && !info[1]->IsNull())
-    worker->SaveToPersistent("local_path", info[1]->ToObject());
+    worker->SaveToPersistent("local_path", Nan::To<v8::Object>(info[1]).ToLocalChecked());
   if (!info[2]->IsUndefined() && !info[2]->IsNull())
-    worker->SaveToPersistent("options", info[2]->ToObject());
+    worker->SaveToPersistent("options", Nan::To<v8::Object>(info[2]).ToLocalChecked());
 
   AsyncLibgit2QueueWorker(worker);
   return;
@@ -140,9 +140,9 @@ void GitClone::CloneWorker::HandleOKCallback() {
     if (baton->error) {
       v8::Local<v8::Object> err;
       if (baton->error->message) {
-        err = Nan::Error(baton->error->message)->ToObject();
+        err = Nan::To<v8::Object>(Nan::Error(baton->error->message)).ToLocalChecked();
       } else {
-        err = Nan::Error("Method clone has thrown an error.")->ToObject();
+        err = Nan::To<v8::Object>(Nan::Error("Method clone has thrown an error.")).ToLocalChecked();
       }
       err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
       err->Set(Nan::New("errorFunction").ToLocalChecked(),
@@ -168,13 +168,13 @@ void GitClone::CloneWorker::HandleOKCallback() {
           continue;
         }
 
-        v8::Local<v8::Object> nodeObj = node->ToObject();
+        v8::Local<v8::Object> nodeObj = Nan::To<v8::Object>(node).ToLocalChecked();
         v8::Local<v8::Value> checkValue = GetPrivate(
             nodeObj, Nan::New("NodeGitPromiseError").ToLocalChecked());
 
         if (!checkValue.IsEmpty() && !checkValue->IsNull() &&
             !checkValue->IsUndefined()) {
-          v8::Local<v8::Value> argv[1] = {checkValue->ToObject()};
+          v8::Local<v8::Value> argv[1] = {Nan::To<v8::Object>(checkValue).ToLocalChecked()};
           callback->Call(1, argv, async_resource);
           callbackFired = true;
           break;
@@ -184,7 +184,7 @@ void GitClone::CloneWorker::HandleOKCallback() {
         for (unsigned int propIndex = 0; propIndex < properties->Length();
              ++propIndex) {
           v8::Local<v8::String> propName =
-              properties->Get(propIndex)->ToString();
+              Nan::To<v8::String>(properties->Get(propIndex)).ToLocalChecked();
           v8::Local<v8::Value> nodeToQueue = nodeObj->Get(propName);
           if (!nodeToQueue->IsUndefined()) {
             workerArguments.push(nodeToQueue);
@@ -194,7 +194,7 @@ void GitClone::CloneWorker::HandleOKCallback() {
 
       if (!callbackFired) {
         v8::Local<v8::Object> err =
-            Nan::Error("Method clone has thrown an error.")->ToObject();
+            Nan::To<v8::Object>(Nan::Error("Method clone has thrown an error.")).ToLocalChecked();
         err->Set(Nan::New("errno").ToLocalChecked(),
                  Nan::New(baton->error_code));
         err->Set(Nan::New("errorFunction").ToLocalChecked(),
diff --git a/generate/templates/manual/commit/extract_signature.cc b/generate/templates/manual/commit/extract_signature.cc
index c17749539..ebcb78fe0 100644
--- a/generate/templates/manual/commit/extract_signature.cc
+++ b/generate/templates/manual/commit/extract_signature.cc
@@ -28,11 +28,11 @@ NAN_METHOD(GitCommit::ExtractSignature)
   baton->error = NULL;
   baton->signature = GIT_BUF_INIT_CONST(NULL, 0);
   baton->signed_data = GIT_BUF_INIT_CONST(NULL, 0);
-  baton->repo = Nan::ObjectWrap::Unwrap<GitRepository>(info[0]->ToObject())->GetValue();
+  baton->repo = Nan::ObjectWrap::Unwrap<GitRepository>(Nan::To<v8::Object>(info[0]).ToLocalChecked())->GetValue();
 
   // baton->commit_id
   if (info[1]->IsString()) {
-    String::Utf8Value oidString(info[1]->ToString());
+   Nan::Utf8String oidString(Nan::To<v8::String>(info[1]).ToLocalChecked());
     baton->commit_id = (git_oid *)malloc(sizeof(git_oid));
     if (git_oid_fromstr(baton->commit_id, (const char *)strdup(*oidString)) != GIT_OK) {
       free(baton->commit_id);
@@ -44,12 +44,12 @@ NAN_METHOD(GitCommit::ExtractSignature)
       }
     }
   } else {
-    baton->commit_id = Nan::ObjectWrap::Unwrap<GitOid>(info[1]->ToObject())->GetValue();
+    baton->commit_id = Nan::ObjectWrap::Unwrap<GitOid>(Nan::To<v8::Object>(info[1]).ToLocalChecked())->GetValue();
   }
 
   // baton->field
   if (info[2]->IsString()) {
-    String::Utf8Value field(info[2]->ToString());
+   Nan::Utf8String field(Nan::To<v8::String>(info[2]).ToLocalChecked());
     baton->field = (char *)malloc(field.length() + 1);
     memcpy((void *)baton->field, *field, field.length());
     baton->field[field.length()] = 0;
@@ -65,8 +65,8 @@ NAN_METHOD(GitCommit::ExtractSignature)
   }
 
   ExtractSignatureWorker *worker = new ExtractSignatureWorker(baton, callback);
-  worker->SaveToPersistent("repo", info[0]->ToObject());
-  worker->SaveToPersistent("commit_id", info[1]->ToObject());
+  worker->SaveToPersistent("repo", Nan::To<v8::Object>(info[0]).ToLocalChecked());
+  worker->SaveToPersistent("commit_id", Nan::To<v8::Object>(info[1]).ToLocalChecked());
   Nan::AsyncQueueWorker(worker);
   return;
 }
@@ -132,7 +132,7 @@ void GitCommit::ExtractSignatureWorker::HandleOKCallback()
   }
   else if (baton->error_code < 0)
   {
-    Local<v8::Object> err = Nan::Error("Extract Signature has thrown an error.")->ToObject();
+    Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Extract Signature has thrown an error.")).ToLocalChecked();
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Commit.extractSignature").ToLocalChecked());
     Local<v8::Value> argv[1] = {
diff --git a/generate/templates/manual/filter_list/load.cc b/generate/templates/manual/filter_list/load.cc
index 6075fd59c..9a3e1ba25 100644
--- a/generate/templates/manual/filter_list/load.cc
+++ b/generate/templates/manual/filter_list/load.cc
@@ -47,14 +47,14 @@ NAN_METHOD(GitFilterList::Load) {
   // start convert_from_v8 block
   git_repository *from_repo = NULL;
   from_repo =
-      Nan::ObjectWrap::Unwrap<GitRepository>(info[0]->ToObject())->GetValue();
+      Nan::ObjectWrap::Unwrap<GitRepository>(Nan::To<v8::Object>(info[0]).ToLocalChecked())->GetValue();
   // end convert_from_v8 block
   baton->repo = from_repo;
   // start convert_from_v8 block
   git_blob *from_blob = NULL;
   if (info[1]->IsObject()) {
     from_blob =
-        Nan::ObjectWrap::Unwrap<GitBlob>(info[1]->ToObject())->GetValue();
+        Nan::ObjectWrap::Unwrap<GitBlob>(Nan::To<v8::Object>(info[1]).ToLocalChecked())->GetValue();
   } else {
     from_blob = 0;
   }
@@ -63,7 +63,7 @@ NAN_METHOD(GitFilterList::Load) {
   // start convert_from_v8 block
   const char *from_path = NULL;
 
-  String::Utf8Value path(info[2]->ToString());
+ Nan::Utf8String path(Nan::To<v8::String>(info[2]).ToLocalChecked());
   // malloc with one extra byte so we can add the terminating null character
   // C-strings expect:
   from_path = (const char *)malloc(path.length() + 1);
@@ -93,15 +93,15 @@ NAN_METHOD(GitFilterList::Load) {
   LoadWorker *worker = new LoadWorker(baton, callback);
 
   if (!info[0]->IsUndefined() && !info[0]->IsNull())
-    worker->SaveToPersistent("repo", info[0]->ToObject());
+    worker->SaveToPersistent("repo", Nan::To<v8::Object>(info[0]).ToLocalChecked());
   if (!info[1]->IsUndefined() && !info[1]->IsNull())
-    worker->SaveToPersistent("blob", info[1]->ToObject());
+    worker->SaveToPersistent("blob", Nan::To<v8::Object>(info[1]).ToLocalChecked());
   if (!info[2]->IsUndefined() && !info[2]->IsNull())
-    worker->SaveToPersistent("path", info[2]->ToObject());
+    worker->SaveToPersistent("path", Nan::To<v8::Object>(info[2]).ToLocalChecked());
   if (!info[3]->IsUndefined() && !info[3]->IsNull())
-    worker->SaveToPersistent("mode", info[3]->ToObject());
+    worker->SaveToPersistent("mode", Nan::To<v8::Object>(info[3]).ToLocalChecked());
   if (!info[4]->IsUndefined() && !info[4]->IsNull())
-    worker->SaveToPersistent("flags", info[4]->ToObject());
+    worker->SaveToPersistent("flags", Nan::To<v8::Object>(info[4]).ToLocalChecked());
 
   AsyncLibgit2QueueWorker(worker);
   return;
@@ -139,12 +139,12 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
       Nan::Set(
         owners,
         Nan::New<Number>(0),
-        this->GetFromPersistent("repo")->ToObject()
+        Nan::To<v8::Object>(this->GetFromPersistent("repo")).ToLocalChecked()
       );
 
       for (uint32_t index = 0; index < propertyNames->Length(); ++index) {
-        v8::Local<v8::String> propertyName = propertyNames->Get(index)->ToString();
-        String::Utf8Value propertyNameAsUtf8Value(propertyName);
+        v8::Local<v8::String> propertyName = Nan::To<v8::String>(propertyNames->Get(index)).ToLocalChecked();
+       Nan::Utf8String propertyNameAsUtf8Value(propertyName);
         const char *propertyNameAsCString = *propertyNameAsUtf8Value;
 
         bool isNotMethodOnRegistry = strcmp("register", propertyNameAsCString)
@@ -158,7 +158,7 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
         }
       }
 
-      to = GitFilterList::New(baton->filters, true, owners->ToObject());
+      to = GitFilterList::New(baton->filters, true, Nan::To<v8::Object>(owners).ToLocalChecked());
     } else {
       to = Nan::Null();
     }
@@ -172,9 +172,9 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
     if (baton->error) {
       v8::Local<v8::Object> err;
       if (baton->error->message) {
-        err = Nan::Error(baton->error->message)->ToObject();
+        err = Nan::To<v8::Object>(Nan::Error(baton->error->message)).ToLocalChecked();
       } else {
-        err = Nan::Error("Method load has thrown an error.")->ToObject();
+        err = Nan::To<v8::Object>(Nan::Error("Method load has thrown an error.")).ToLocalChecked();
       }
       err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
       err->Set(Nan::New("errorFunction").ToLocalChecked(),
@@ -202,13 +202,13 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
           continue;
         }
 
-        v8::Local<v8::Object> nodeObj = node->ToObject();
+        v8::Local<v8::Object> nodeObj = Nan::To<v8::Object>(node).ToLocalChecked();
         v8::Local<v8::Value> checkValue = GetPrivate(
             nodeObj, Nan::New("NodeGitPromiseError").ToLocalChecked());
 
         if (!checkValue.IsEmpty() && !checkValue->IsNull() &&
             !checkValue->IsUndefined()) {
-          v8::Local<v8::Value> argv[1] = {checkValue->ToObject()};
+          v8::Local<v8::Value> argv[1] = {Nan::To<v8::Object>(checkValue).ToLocalChecked()};
           callback->Call(1, argv, async_resource);
           callbackFired = true;
           break;
@@ -218,7 +218,7 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
         for (unsigned int propIndex = 0; propIndex < properties->Length();
              ++propIndex) {
           v8::Local<v8::String> propName =
-              properties->Get(propIndex)->ToString();
+              Nan::To<v8::String>(properties->Get(propIndex)).ToLocalChecked();
           v8::Local<v8::Value> nodeToQueue = nodeObj->Get(propName);
           if (!nodeToQueue->IsUndefined()) {
             workerArguments.push(nodeToQueue);
@@ -228,7 +228,7 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
 
       if (!callbackFired) {
         v8::Local<v8::Object> err =
-            Nan::Error("Method load has thrown an error.")->ToObject();
+            Nan::To<v8::Object>(Nan::Error("Method load has thrown an error.")).ToLocalChecked();
         err->Set(Nan::New("errno").ToLocalChecked(),
                  Nan::New(baton->error_code));
         err->Set(Nan::New("errorFunction").ToLocalChecked(),
diff --git a/generate/templates/manual/filter_source/repo.cc b/generate/templates/manual/filter_source/repo.cc
index cf9c1a833..f35fb33a4 100644
--- a/generate/templates/manual/filter_source/repo.cc
+++ b/generate/templates/manual/filter_source/repo.cc
@@ -60,9 +60,9 @@ void GitFilterSource::RepoWorker::HandleOKCallback() {
     if (baton->error) {
       v8::Local<v8::Object> err;
       if (baton->error->message) {
-        err = Nan::Error(baton->error->message)->ToObject();
+        err = Nan::To<v8::Object>(Nan::Error(baton->error->message)).ToLocalChecked();
       } else {
-        err = Nan::Error("Method repo has thrown an error.")->ToObject();
+        err = Nan::To<v8::Object>(Nan::Error("Method repo has thrown an error.")).ToLocalChecked();
       }
       err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
       err->Set(Nan::New("errorFunction").ToLocalChecked(),
@@ -74,7 +74,7 @@ void GitFilterSource::RepoWorker::HandleOKCallback() {
       free((void *)baton->error);
     } else if (baton->error_code < 0) {
       v8::Local<v8::Object> err =
-          Nan::Error("Method repo has thrown an error.")->ToObject();
+          Nan::To<v8::Object>(Nan::Error("Method repo has thrown an error.")).ToLocalChecked();
       err->Set(Nan::New("errno").ToLocalChecked(),
                Nan::New(baton->error_code));
       err->Set(Nan::New("errorFunction").ToLocalChecked(),
diff --git a/generate/templates/manual/patches/convenient_patches.cc b/generate/templates/manual/patches/convenient_patches.cc
index cc108a8ae..ef9322a43 100644
--- a/generate/templates/manual/patches/convenient_patches.cc
+++ b/generate/templates/manual/patches/convenient_patches.cc
@@ -12,7 +12,7 @@ NAN_METHOD(GitPatch::ConvenientFromDiff) {
   baton->error_code = GIT_OK;
   baton->error = NULL;
 
-  baton->diff = Nan::ObjectWrap::Unwrap<GitDiff>(info[0]->ToObject())->GetValue();
+  baton->diff = Nan::ObjectWrap::Unwrap<GitDiff>(Nan::To<v8::Object>(info[0]).ToLocalChecked())->GetValue();
   baton->out = new std::vector<PatchData *>;
   baton->out->reserve(git_diff_num_deltas(baton->diff));
 
@@ -97,9 +97,9 @@ void GitPatch::ConvenientFromDiffWorker::HandleOKCallback() {
   if (baton->error) {
     Local<v8::Object> err;
     if (baton->error->message) {
-      err = Nan::Error(baton->error->message)->ToObject();
+      err = Nan::To<v8::Object>(Nan::Error(baton->error->message)).ToLocalChecked();
     } else {
-      err = Nan::Error("Method convenientFromDiff has thrown an error.")->ToObject();
+      err = Nan::To<v8::Object>(Nan::Error("Method convenientFromDiff has thrown an error.")).ToLocalChecked();
     }
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Patch.convenientFromDiff").ToLocalChecked());
@@ -118,7 +118,7 @@ void GitPatch::ConvenientFromDiffWorker::HandleOKCallback() {
   }
 
   if (baton->error_code < 0) {
-    Local<v8::Object> err = Nan::Error("method convenientFromDiff has thrown an error.")->ToObject();
+    Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("method convenientFromDiff has thrown an error.")).ToLocalChecked();
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Patch.convenientFromDiff").ToLocalChecked());
     Local<v8::Value> argv[1] = {
diff --git a/generate/templates/manual/remote/ls.cc b/generate/templates/manual/remote/ls.cc
index f81728256..9fccf4af3 100644
--- a/generate/templates/manual/remote/ls.cc
+++ b/generate/templates/manual/remote/ls.cc
@@ -85,7 +85,7 @@ void GitRemote::ReferenceListWorker::HandleOKCallback()
   }
   else if (baton->error_code < 0)
   {
-    Local<v8::Object> err = Nan::Error("Reference List has thrown an error.")->ToObject();
+    Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Reference List has thrown an error.")).ToLocalChecked();
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Remote.referenceList").ToLocalChecked());
     Local<v8::Value> argv[1] = {
diff --git a/generate/templates/manual/repository/get_references.cc b/generate/templates/manual/repository/get_references.cc
index d0e4fd987..910352b52 100644
--- a/generate/templates/manual/repository/get_references.cc
+++ b/generate/templates/manual/repository/get_references.cc
@@ -90,7 +90,7 @@ void GitRepository::GetReferencesWorker::HandleOKCallback()
         GitRefs::New(
           reference,
           true,
-          GitRepository::New(git_reference_owner(reference), true)->ToObject()
+          Nan::To<v8::Object>(GitRepository::New(git_reference_owner(reference), true)).ToLocalChecked()
         )
       );
     }
@@ -118,7 +118,7 @@ void GitRepository::GetReferencesWorker::HandleOKCallback()
   }
   else if (baton->error_code < 0)
   {
-    Local<v8::Object> err = Nan::Error("Repository getReferences has thrown an error.")->ToObject();
+    Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Repository getReferences has thrown an error.")).ToLocalChecked();
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.getReferences").ToLocalChecked());
     Local<v8::Value> argv[1] = {
diff --git a/generate/templates/manual/repository/get_remotes.cc b/generate/templates/manual/repository/get_remotes.cc
index e16f1131c..457ea3ebe 100644
--- a/generate/templates/manual/repository/get_remotes.cc
+++ b/generate/templates/manual/repository/get_remotes.cc
@@ -91,7 +91,7 @@ void GitRepository::GetRemotesWorker::HandleOKCallback()
         GitRemote::New(
           remote,
           true,
-          GitRepository::New(git_remote_owner(remote), true)->ToObject()
+          Nan::To<v8::Object>(GitRepository::New(git_remote_owner(remote), true)).ToLocalChecked()
         )
       );
     }
@@ -119,7 +119,7 @@ void GitRepository::GetRemotesWorker::HandleOKCallback()
   }
   else if (baton->error_code < 0)
   {
-    Local<v8::Object> err = Nan::Error("Repository refreshRemotes has thrown an error.")->ToObject();
+    Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Repository refreshRemotes has thrown an error.")).ToLocalChecked();
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.refreshRemotes").ToLocalChecked());
     Local<v8::Value> argv[1] = {
diff --git a/generate/templates/manual/repository/get_submodules.cc b/generate/templates/manual/repository/get_submodules.cc
index d51d0a0fd..0d73424b1 100644
--- a/generate/templates/manual/repository/get_submodules.cc
+++ b/generate/templates/manual/repository/get_submodules.cc
@@ -72,7 +72,7 @@ void GitRepository::GetSubmodulesWorker::HandleOKCallback()
         GitSubmodule::New(
           submodule,
           true,
-          GitRepository::New(git_submodule_owner(submodule), true)->ToObject()
+          Nan::To<v8::Object>(GitRepository::New(git_submodule_owner(submodule), true)).ToLocalChecked()
         )
       );
     }
@@ -100,7 +100,7 @@ void GitRepository::GetSubmodulesWorker::HandleOKCallback()
   }
   else if (baton->error_code < 0)
   {
-    Local<v8::Object> err = Nan::Error("Repository getSubmodules has thrown an error.")->ToObject();
+    Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Repository getSubmodules has thrown an error.")).ToLocalChecked();
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.getSubmodules").ToLocalChecked());
     Local<v8::Value> argv[1] = {
diff --git a/generate/templates/manual/repository/refresh_references.cc b/generate/templates/manual/repository/refresh_references.cc
index 65548812c..8db164fb5 100644
--- a/generate/templates/manual/repository/refresh_references.cc
+++ b/generate/templates/manual/repository/refresh_references.cc
@@ -206,7 +206,7 @@ class RefreshedRefModel {
       // the destructor didn't double free, but that still segfaulted internally in Node.
       v8::Local<v8::Object> buffer = Nan::CopyBuffer(tagOdbBuffer, tagOdbBufferLength).ToLocalChecked();
       v8::Local<v8::Value> toStringProp = Nan::Get(buffer, Nan::New("toString").ToLocalChecked()).ToLocalChecked();
-      v8::Local<v8::Object> jsTagOdbObjectString = Nan::CallAsFunction(toStringProp->ToObject(), buffer, 0, NULL).ToLocalChecked()->ToObject();
+      v8::Local<v8::Object> jsTagOdbObjectString = Nan::To<v8::Object>(Nan::CallAsFunction(Nan::To<v8::Object>(toStringProp).ToLocalChecked(), buffer, 0, NULL).ToLocalChecked()).ToLocalChecked();
 
       v8::Local<v8::Object> _signatureRegexesBySignatureType = Nan::New(signatureRegexesBySignatureType);
       v8::Local<v8::Array> signatureRegexes = v8::Local<v8::Array>::Cast(Nan::Get(_signatureRegexesBySignatureType, signatureType).ToLocalChecked());
@@ -217,9 +217,9 @@ class RefreshedRefModel {
         };
 
         v8::Local<v8::Value> matchProp = Nan::Get(jsTagOdbObjectString, Nan::New("match").ToLocalChecked()).ToLocalChecked();
-        v8::Local<v8::Value> match = Nan::CallAsFunction(matchProp->ToObject(), jsTagOdbObjectString, 1, argv).ToLocalChecked();
+        v8::Local<v8::Value> match = Nan::CallAsFunction(Nan::To<v8::Object>(matchProp).ToLocalChecked(), jsTagOdbObjectString, 1, argv).ToLocalChecked();
         if (match->IsArray()) {
-          jsTagSignature = Nan::Get(match->ToObject(), 0).ToLocalChecked();
+          jsTagSignature = Nan::Get(Nan::To<v8::Object>(match).ToLocalChecked(), 0).ToLocalChecked();
           break;
         }
       }
@@ -379,7 +379,7 @@ NAN_METHOD(GitRepository::RefreshReferences)
       return Nan::ThrowError("Signature type must be \"gpgsig\" or \"x509\".");
     }
 
-    v8::Local<v8::String> signatureTypeParam = info[0]->ToString();
+    v8::Local<v8::String> signatureTypeParam = Nan::To<v8::String>(info[0]).ToLocalChecked();
     if (
       Nan::Equals(signatureTypeParam, Nan::New("gpgsig").ToLocalChecked()) != Nan::Just(true)
       && Nan::Equals(signatureTypeParam, Nan::New("x509").ToLocalChecked()) != Nan::Just(true)
@@ -611,7 +611,7 @@ void GitRepository::RefreshReferencesWorker::HandleOKCallback()
       Nan::New<String>(refreshData->headRefFullName).ToLocalChecked()
     );
 
-    v8::Local<v8::String> signatureType = GetFromPersistent("signatureType")->ToString();
+    v8::Local<v8::String> signatureType = Nan::To<v8::String>(GetFromPersistent("signatureType")).ToLocalChecked();
 
     unsigned int numRefs = refreshData->refs.size();
     v8::Local<v8::Array> refs = Nan::New<v8::Array>(numRefs);
@@ -672,7 +672,7 @@ void GitRepository::RefreshReferencesWorker::HandleOKCallback()
   }
   else if (baton->error_code < 0)
   {
-    Local<v8::Object> err = Nan::Error("Repository refreshReferences has thrown an error.")->ToObject();
+    Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Repository refreshReferences has thrown an error.")).ToLocalChecked();
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.refreshReferences").ToLocalChecked());
     Local<v8::Value> argv[1] = {
diff --git a/generate/templates/manual/revwalk/commit_walk.cc b/generate/templates/manual/revwalk/commit_walk.cc
index af0ff112a..1c7bed6fc 100644
--- a/generate/templates/manual/revwalk/commit_walk.cc
+++ b/generate/templates/manual/revwalk/commit_walk.cc
@@ -87,7 +87,7 @@ void GitRevwalk::CommitWalkWorker::HandleOKCallback() {
         GitCommit::New(
           commit,
           true,
-          GitRepository::New(git_commit_owner(commit), true)->ToObject()
+          Nan::To<v8::Object>(GitRepository::New(git_commit_owner(commit), true)).ToLocalChecked()
         )
       );
     }
@@ -110,7 +110,7 @@ void GitRevwalk::CommitWalkWorker::HandleOKCallback() {
 
     free((void *)baton->error);
   } else if (baton->error_code < 0) {
-    Local<v8::Object> err = Nan::Error("Revwalk commitWalk has thrown an error.")->ToObject();
+    Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Revwalk commitWalk has thrown an error.")).ToLocalChecked();
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.commitWalk").ToLocalChecked());
     Local<v8::Value> argv[1] = {
diff --git a/generate/templates/manual/revwalk/fast_walk.cc b/generate/templates/manual/revwalk/fast_walk.cc
index fbf5c09b5..619f8c6c8 100644
--- a/generate/templates/manual/revwalk/fast_walk.cc
+++ b/generate/templates/manual/revwalk/fast_walk.cc
@@ -90,9 +90,9 @@ void GitRevwalk::FastWalkWorker::HandleOKCallback()
     {
       Local<v8::Object> err;
       if (baton->error->message) {
-        err = Nan::Error(baton->error->message)->ToObject();
+        err = Nan::To<v8::Object>(Nan::Error(baton->error->message)).ToLocalChecked();
       } else {
-        err = Nan::Error("Method fastWalk has thrown an error.")->ToObject();
+        err = Nan::To<v8::Object>(Nan::Error("Method fastWalk has thrown an error.")).ToLocalChecked();
       }
       err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
       err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fastWalk").ToLocalChecked());
@@ -131,13 +131,13 @@ void GitRevwalk::FastWalkWorker::HandleOKCallback()
           continue;
         }
 
-        Local<v8::Object> nodeObj = node->ToObject();
+        Local<v8::Object> nodeObj = Nan::To<v8::Object>(node).ToLocalChecked();
         Local<v8::Value> checkValue = GetPrivate(nodeObj, Nan::New("NodeGitPromiseError").ToLocalChecked());
 
         if (!checkValue.IsEmpty() && !checkValue->IsNull() && !checkValue->IsUndefined())
         {
           Local<v8::Value> argv[1] = {
-            checkValue->ToObject()
+            Nan::To<v8::Object>(checkValue).ToLocalChecked()
           };
           callback->Call(1, argv, async_resource);
           callbackFired = true;
@@ -147,7 +147,7 @@ void GitRevwalk::FastWalkWorker::HandleOKCallback()
         Local<v8::Array> properties = nodeObj->GetPropertyNames();
         for (unsigned int propIndex = 0; propIndex < properties->Length(); ++propIndex)
         {
-          Local<v8::String> propName = properties->Get(propIndex)->ToString();
+          Local<v8::String> propName = Nan::To<v8::String>(properties->Get(propIndex)).ToLocalChecked();
           Local<v8::Value> nodeToQueue = nodeObj->Get(propName);
           if (!nodeToQueue->IsUndefined())
           {
@@ -158,7 +158,7 @@ void GitRevwalk::FastWalkWorker::HandleOKCallback()
 
       if (!callbackFired)
       {
-        Local<v8::Object> err = Nan::Error("Method next has thrown an error.")->ToObject();
+        Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method next has thrown an error.")).ToLocalChecked();
         err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
         err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fastWalk").ToLocalChecked());
         Local<v8::Value> argv[1] = {
diff --git a/generate/templates/manual/revwalk/file_history_walk.cc b/generate/templates/manual/revwalk/file_history_walk.cc
index 49c23445e..f7cec692a 100644
--- a/generate/templates/manual/revwalk/file_history_walk.cc
+++ b/generate/templates/manual/revwalk/file_history_walk.cc
@@ -34,10 +34,10 @@ class FileHistoryEvent {
     Nan::Set(
       owners,
       Nan::New<v8::Number>(owners->Length()),
-      GitRepository::New(
+      Nan::To<v8::Object>(GitRepository::New(
         git_commit_owner(commit),
         true
-      )->ToObject()
+      )).ToLocalChecked()
     );
     Nan::Set(historyEntry, Nan::New("commit").ToLocalChecked(), GitCommit::New(commit, true, owners));
     commit = NULL;
@@ -196,7 +196,7 @@ NAN_METHOD(GitRevwalk::FileHistoryWalk)
 
   baton->error_code = GIT_OK;
   baton->error = NULL;
-  String::Utf8Value from_js_file_path(info[0]->ToString());
+ Nan::Utf8String from_js_file_path(Nan::To<v8::String>(info[0]).ToLocalChecked());
   baton->file_path = strdup(*from_js_file_path);
   baton->max_count = Nan::To<unsigned int>(info[1]).FromJust();
   baton->out = new std::vector<void *>;
@@ -444,9 +444,9 @@ void GitRevwalk::FileHistoryWalkWorker::HandleOKCallback()
   if (baton->error) {
     v8::Local<v8::Object> err;
     if (baton->error->message) {
-      err = Nan::Error(baton->error->message)->ToObject();
+      err = Nan::To<v8::Object>(Nan::Error(baton->error->message)).ToLocalChecked();
     } else {
-      err = Nan::Error("Method fileHistoryWalk has thrown an error.")->ToObject();
+      err = Nan::To<v8::Object>(Nan::Error("Method fileHistoryWalk has thrown an error.")).ToLocalChecked();
     }
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fileHistoryWalk").ToLocalChecked());
@@ -464,7 +464,7 @@ void GitRevwalk::FileHistoryWalkWorker::HandleOKCallback()
   }
 
   if (baton->error_code < 0) {
-    v8::Local<v8::Object> err = Nan::Error("Method next has thrown an error.")->ToObject();
+    v8::Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method next has thrown an error.")).ToLocalChecked();
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fileHistoryWalk").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
diff --git a/generate/templates/manual/src/filter_registry.cc b/generate/templates/manual/src/filter_registry.cc
index 410255289..62de8f80a 100644
--- a/generate/templates/manual/src/filter_registry.cc
+++ b/generate/templates/manual/src/filter_registry.cc
@@ -54,8 +54,8 @@ NAN_METHOD(GitFilterRegistry::GitFilterRegister) {
 
   FilterRegisterBaton *baton = new FilterRegisterBaton;
 
-  baton->filter = Nan::ObjectWrap::Unwrap<GitFilter>(info[1]->ToObject())->GetValue();
-  String::Utf8Value name(info[0]->ToString());
+  baton->filter = Nan::ObjectWrap::Unwrap<GitFilter>(Nan::To<v8::Object>(info[1]).ToLocalChecked())->GetValue();
+ Nan::Utf8String name(Nan::To<v8::String>(info[0]).ToLocalChecked());
 
   baton->filter_name = (char *)malloc(name.length() + 1);
   memcpy((void *)baton->filter_name, *name, name.length());
@@ -64,13 +64,13 @@ NAN_METHOD(GitFilterRegistry::GitFilterRegister) {
   baton->error_code = GIT_OK;
   baton->filter_priority = Nan::To<int>(info[2]).FromJust();
 
-  Nan::New(GitFilterRegistry::persistentHandle)->Set(info[0]->ToString(), info[1]->ToObject());
+  Nan::New(GitFilterRegistry::persistentHandle)->Set(Nan::To<v8::Object>(Nan::To<v8::String>(info[0]).ToLocalChecked(), info[1]).ToLocalChecked());
 
   Nan::Callback *callback = new Nan::Callback(Local<Function>::Cast(info[3]));
   RegisterWorker *worker = new RegisterWorker(baton, callback);
 
-  worker->SaveToPersistent("filter_name", info[0]->ToObject());
-  worker->SaveToPersistent("filter_priority", info[2]->ToObject());
+  worker->SaveToPersistent("filter_name", Nan::To<v8::Object>(info[0]).ToLocalChecked());
+  worker->SaveToPersistent("filter_priority", Nan::To<v8::Object>(info[2]).ToLocalChecked());
 
   AsyncLibgit2QueueWorker(worker);
   return;
@@ -102,9 +102,9 @@ void GitFilterRegistry::RegisterWorker::HandleOKCallback() {
   else if (baton->error) {
     v8::Local<v8::Object> err;
     if (baton->error->message) {
-      err = Nan::Error(baton->error->message)->ToObject();
+      err = Nan::To<v8::Object>(Nan::Error(baton->error->message)).ToLocalChecked();
     } else {
-      err = Nan::Error("Method register has thrown an error.")->ToObject();
+      err = Nan::To<v8::Object>(Nan::Error("Method register has thrown an error.")).ToLocalChecked();
     }
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.register").ToLocalChecked());
@@ -117,7 +117,7 @@ void GitFilterRegistry::RegisterWorker::HandleOKCallback() {
     free((void *)baton->error);
   }
   else if (baton->error_code < 0) {
-    v8::Local<v8::Object> err = Nan::Error("Method register has thrown an error.")->ToObject();
+    v8::Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method register has thrown an error.")).ToLocalChecked();
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.register").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
@@ -144,7 +144,7 @@ NAN_METHOD(GitFilterRegistry::GitFilterUnregister) {
   }
 
   FilterUnregisterBaton *baton = new FilterUnregisterBaton;
-  String::Utf8Value name(info[0]->ToString());
+ Nan::Utf8String name(Nan::To<v8::String>(info[0]).ToLocalChecked());
 
   baton->filter_name = (char *)malloc(name.length() + 1);
   memcpy((void *)baton->filter_name, *name, name.length());
@@ -188,9 +188,9 @@ void GitFilterRegistry::UnregisterWorker::HandleOKCallback() {
   else if (baton->error) {
     v8::Local<v8::Object> err;
     if (baton->error->message) {
-      err = Nan::Error(baton->error->message)->ToObject();
+      err = Nan::To<v8::Object>(Nan::Error(baton->error->message)).ToLocalChecked();
     } else {
-      err = Nan::Error("Method register has thrown an error.")->ToObject();
+      err = Nan::To<v8::Object>(Nan::Error("Method register has thrown an error.")).ToLocalChecked();
     }
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.unregister").ToLocalChecked());
@@ -203,7 +203,7 @@ void GitFilterRegistry::UnregisterWorker::HandleOKCallback() {
     free((void *)baton->error);
   }
   else if (baton->error_code < 0) {
-    v8::Local<v8::Object> err = Nan::Error("Method unregister has thrown an error.")->ToObject();
+    v8::Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method unregister has thrown an error.")).ToLocalChecked();
     err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
     err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.unregister").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
diff --git a/generate/templates/manual/src/git_buf_converter.cc b/generate/templates/manual/src/git_buf_converter.cc
index 0c1969504..1413168af 100644
--- a/generate/templates/manual/src/git_buf_converter.cc
+++ b/generate/templates/manual/src/git_buf_converter.cc
@@ -10,7 +10,7 @@ using namespace node;
 
 git_buf *GitBufConverter::Convert(Local<v8::Value> val) {
   if (val->IsString() || val->IsStringObject()) {
-    v8::String::Utf8Value param1(val->ToString());
+    v8::String::Utf8Value param1(Nan::To<v8::String>(val).ToLocalChecked());
     std::string v8String = std::string(*param1);
 
     const size_t size = sizeof(git_buf);
diff --git a/generate/templates/manual/src/nodegit_wrapper.cc b/generate/templates/manual/src/nodegit_wrapper.cc
index 26ead60da..e9fb8ebdb 100644
--- a/generate/templates/manual/src/nodegit_wrapper.cc
+++ b/generate/templates/manual/src/nodegit_wrapper.cc
@@ -67,7 +67,7 @@ NAN_METHOD(NodeGitWrapper<Traits>::JSNewFunction) {
     instance = new cppClass(static_cast<cType *>(
       Local<External>::Cast(info[0])->Value()),
       Nan::To<bool>(info[1]).FromJust(),
-      info.Length() >= 3 && !info[2].IsEmpty() && info[2]->IsObject() ? info[2]->ToObject() : Local<v8::Object>()
+      info.Length() >= 3 && !Nan::To<v8::Object>(info[2].IsEmpty() && info[2]->IsObject() ? info[2]).ToLocalChecked() : Local<v8::Object>()
     );
   }
 
diff --git a/generate/templates/manual/src/promise_completion.cc b/generate/templates/manual/src/promise_completion.cc
index fd34dfa77..88bde47da 100644
--- a/generate/templates/manual/src/promise_completion.cc
+++ b/generate/templates/manual/src/promise_completion.cc
@@ -9,10 +9,10 @@ Nan::Persistent<v8::Function> PromiseCompletion::promiseRejected;
 void PromiseCompletion::InitializeComponent() {
   v8::Local<v8::FunctionTemplate> newTemplate = Nan::New<v8::FunctionTemplate>(New);
   newTemplate->InstanceTemplate()->SetInternalFieldCount(1);
-  newFn.Reset(newTemplate->GetFunction());
+  newFn.Reset(Nan::GetFunction(newTemplate).ToLocalChecked());
 
-  promiseFulfilled.Reset(Nan::New<v8::FunctionTemplate>(PromiseFulfilled)->GetFunction());
-  promiseRejected.Reset(Nan::New<v8::FunctionTemplate>(PromiseRejected)->GetFunction());
+  promiseFulfilled.Reset(Nan::GetFunction(Nan::New<v8::FunctionTemplate>(PromiseFulfilled)).ToLocalChecked());
+  promiseRejected.Reset(Nan::GetFunction(Nan::New<v8::FunctionTemplate>(PromiseRejected)).ToLocalChecked());
 }
 
 bool PromiseCompletion::ForwardIfPromise(v8::Local<v8::Value> result, AsyncBaton *baton, Callback callback)
@@ -21,7 +21,7 @@ bool PromiseCompletion::ForwardIfPromise(v8::Local<v8::Value> result, AsyncBaton
 
   // check if the result is a promise
   if (!result.IsEmpty() && result->IsObject()) {
-    Nan::MaybeLocal<v8::Value> maybeThenProp = Nan::Get(result->ToObject(), Nan::New("then").ToLocalChecked());
+    Nan::MaybeLocal<v8::Value> maybeThenProp = Nan::Get(Nan::To<v8::Object>(result).ToLocalChecked(), Nan::New("then").ToLocalChecked());
     if (!maybeThenProp.IsEmpty()) {
       v8::Local<v8::Value> thenProp = maybeThenProp.ToLocalChecked();
       if(thenProp->IsFunction()) {
@@ -54,7 +54,7 @@ void PromiseCompletion::Setup(v8::Local<v8::Function> thenFn, v8::Local<v8::Valu
   this->callback = callback;
   this->baton = baton;
 
-  v8::Local<v8::Object> promise = result->ToObject();
+  v8::Local<v8::Object> promise = Nan::To<v8::Object>(result).ToLocalChecked();
 
   v8::Local<v8::Object> thisHandle = handle();
 
@@ -78,7 +78,7 @@ v8::Local<v8::Value> PromiseCompletion::Bind(Nan::Persistent<v8::Function> &func
 
   v8::Local<v8::Value> argv[1] = { object };
 
-  return scope.Escape(bind->Call(Nan::New(function), 1, argv));
+  return scope.Escape(Nan::Call(bind, Nan::To<v8::Object>(Nan::New(function)).ToLocalChecked(), 1, argv));
 }
 
 // calls the callback stored in the PromiseCompletion, passing the baton that
@@ -90,7 +90,7 @@ void PromiseCompletion::CallCallback(bool isFulfilled, const Nan::FunctionCallba
     resultOfPromise = info[0];
   }
 
-  PromiseCompletion *promiseCompletion = ObjectWrap::Unwrap<PromiseCompletion>(info.This()->ToObject());
+  PromiseCompletion *promiseCompletion = ObjectWrap::Unwrap<PromiseCompletion>(Nan::To<v8::Object>(info.This()).ToLocalChecked());
 
   (*promiseCompletion->callback)(isFulfilled, promiseCompletion->baton, resultOfPromise);
 }
diff --git a/generate/templates/manual/src/str_array_converter.cc b/generate/templates/manual/src/str_array_converter.cc
index c66f901c3..ed0e93e2b 100644
--- a/generate/templates/manual/src/str_array_converter.cc
+++ b/generate/templates/manual/src/str_array_converter.cc
@@ -17,7 +17,7 @@ git_strarray *StrArrayConverter::Convert(Local<v8::Value> val) {
     return ConvertArray(Array::Cast(*val));
   }
   else if (val->IsString() || val->IsStringObject()) {
-    return ConvertString(val->ToString());
+    return ConvertString(Nan::To<v8::String>(val).ToLocalChecked());
   }
   else {
     return NULL;
diff --git a/generate/templates/partials/async_function.cc b/generate/templates/partials/async_function.cc
index 2fe854bec..ea7c43037 100644
--- a/generate/templates/partials/async_function.cc
+++ b/generate/templates/partials/async_function.cc
@@ -69,7 +69,7 @@ NAN_METHOD({{ cppClassName }}::{{ cppFunctionName }}) {
         worker->SaveToPersistent("{{ arg.name }}", info.This());
       {%elsif not arg.isCallbackFunction %}
         if (!info[{{ arg.jsArg }}]->IsUndefined() && !info[{{ arg.jsArg }}]->IsNull())
-          worker->SaveToPersistent("{{ arg.name }}", info[{{ arg.jsArg }}]->ToObject());
+          worker->SaveToPersistent("{{ arg.name }}", Nan::To<v8::Object>(info[{{ arg.jsArg }}]).ToLocalChecked());
       {%endif%}
     {%endif%}
   {%endeach%}
@@ -163,9 +163,9 @@ void {{ cppClassName }}::{{ cppFunctionName }}Worker::HandleOKCallback() {
     if (baton->error) {
       v8::Local<v8::Object> err;
       if (baton->error->message) {
-        err = Nan::Error(baton->error->message)->ToObject();
+        err = Nan::To<v8::Object>(Nan::Error(baton->error->message)).ToLocalChecked();
       } else {
-        err = Nan::Error("Method {{ jsFunctionName }} has thrown an error.")->ToObject();
+        err = Nan::To<v8::Object>(Nan::Error("Method {{ jsFunctionName }} has thrown an error.")).ToLocalChecked();
       }
       err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
       err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("{{ jsClassName }}.{{ jsFunctionName }}").ToLocalChecked());
@@ -205,12 +205,12 @@ void {{ cppClassName }}::{{ cppFunctionName }}Worker::HandleOKCallback() {
           continue;
         }
 
-        v8::Local<v8::Object> nodeObj = node->ToObject();
+        v8::Local<v8::Object> nodeObj = Nan::To<v8::Object>(node).ToLocalChecked();
         v8::Local<v8::Value> checkValue = GetPrivate(nodeObj, Nan::New("NodeGitPromiseError").ToLocalChecked());
 
         if (!checkValue.IsEmpty() && !checkValue->IsNull() && !checkValue->IsUndefined()) {
           v8::Local<v8::Value> argv[1] = {
-            checkValue->ToObject()
+            Nan::To<v8::Object>(checkValue).ToLocalChecked()
           };
           callback->Call(1, argv, async_resource);
           callbackFired = true;
@@ -219,7 +219,7 @@ void {{ cppClassName }}::{{ cppFunctionName }}Worker::HandleOKCallback() {
 
         v8::Local<v8::Array> properties = nodeObj->GetPropertyNames();
         for (unsigned int propIndex = 0; propIndex < properties->Length(); ++propIndex) {
-          v8::Local<v8::String> propName = properties->Get(propIndex)->ToString();
+          v8::Local<v8::String> propName = Nan::To<v8::String>(properties->Get(propIndex)).ToLocalChecked();
           v8::Local<v8::Value> nodeToQueue = nodeObj->Get(propName);
           if (!nodeToQueue->IsUndefined()) {
             workerArguments.push(nodeToQueue);
@@ -228,7 +228,7 @@ void {{ cppClassName }}::{{ cppFunctionName }}Worker::HandleOKCallback() {
       }
 
       if (!callbackFired) {
-        v8::Local<v8::Object> err = Nan::Error("Method {{ jsFunctionName }} has thrown an error.")->ToObject();
+        v8::Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method {{ jsFunctionName }} has thrown an error.")).ToLocalChecked();
         err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
         err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("{{ jsClassName }}.{{ jsFunctionName }}").ToLocalChecked());
         v8::Local<v8::Value> argv[1] = {
diff --git a/generate/templates/partials/callback_helpers.cc b/generate/templates/partials/callback_helpers.cc
index 7a40aed25..c3810371c 100644
--- a/generate/templates/partials/callback_helpers.cc
+++ b/generate/templates/partials/callback_helpers.cc
@@ -66,7 +66,7 @@ void {{ cppClassName }}::{{ cppFunctionName }}_{{ cbFunction.name }}_async(void
     }
     else if (!result->IsNull() && !result->IsUndefined()) {
       {% if _return.isOutParam %}
-      {{ _return.cppClassName }}* wrapper = Nan::ObjectWrap::Unwrap<{{ _return.cppClassName }}>(result->ToObject());
+      {{ _return.cppClassName }}* wrapper = Nan::ObjectWrap::Unwrap<{{ _return.cppClassName }}>(Nan::To<v8::Object>(result).ToLocalChecked());
       wrapper->selfFreeing = false;
 
       *baton->{{ _return.name }} = wrapper->GetValue();
@@ -100,7 +100,7 @@ void {{ cppClassName }}::{{ cppFunctionName }}_{{ cbFunction.name }}_promiseComp
       }
       else if (!result->IsNull() && !result->IsUndefined()) {
         {% if _return.isOutParam %}
-        {{ _return.cppClassName }}* wrapper = Nan::ObjectWrap::Unwrap<{{ _return.cppClassName }}>(result->ToObject());
+        {{ _return.cppClassName }}* wrapper = Nan::ObjectWrap::Unwrap<{{ _return.cppClassName }}>(Nan::To<v8::Object>(result).ToLocalChecked());
         wrapper->selfFreeing = false;
 
         *baton->{{ _return.name }} = wrapper->GetValue();
diff --git a/generate/templates/partials/convert_from_v8.cc b/generate/templates/partials/convert_from_v8.cc
index c1486d5a6..4f6a146d4 100644
--- a/generate/templates/partials/convert_from_v8.cc
+++ b/generate/templates/partials/convert_from_v8.cc
@@ -19,7 +19,7 @@
     {%endif%}
   {%if cppClassName == 'String'%}
 
-  String::Utf8Value {{ name }}(info[{{ jsArg }}]->ToString());
+ Nan::Utf8String {{ name }}(Nan::To<v8::String>(info[{{ jsArg }}]).ToLocalChecked());
   // malloc with one extra byte so we can add the terminating null character C-strings expect:
   from_{{ name }} = ({{ cType }}) malloc({{ name }}.length() + 1);
   // copy the characters from the nodejs string into our C-string (used instead of strdup or strcpy because nulls in
@@ -36,7 +36,7 @@
   from_{{ name }} = GitBufConverter::Convert(info[{{ jsArg }}]);
   {%elsif cppClassName == 'Wrapper'%}
 
-  String::Utf8Value {{ name }}(info[{{ jsArg }}]->ToString());
+ Nan::Utf8String {{ name }}(Nan::To<v8::String>(info[{{ jsArg }}]).ToLocalChecked());
   // malloc with one extra byte so we can add the terminating null character C-strings expect:
   from_{{ name }} = ({{ cType }}) malloc({{ name }}.length() + 1);
   // copy the characters from the nodejs string into our C-string (used instead of strdup or strcpy because nulls in
@@ -53,12 +53,12 @@
     {%--
       // FIXME: should recursively call convertFromv8.
     --%}
-      from_{{ name }}[i] = Nan::ObjectWrap::Unwrap<{{ arrayElementCppClassName }}>(tmp_{{ name }}->Get(Nan::New(static_cast<double>(i)))->ToObject())->GetValue();
+      from_{{ name }}[i] = Nan::ObjectWrap::Unwrap<{{ arrayElementCppClassName }}>(Nan::To<v8::Object>(tmp_{{ name }}->Get(Nan::New(static_cast<double>(i)))).ToLocalChecked())->GetValue();
       }
   {%elsif cppClassName == 'Function'%}
   {%elsif cppClassName == 'Buffer'%}
 
-  from_{{ name }} = Buffer::Data(info[{{ jsArg }}]->ToObject());
+  from_{{ name }} = Buffer::Data(Nan::To<v8::Object>(info[{{ jsArg }}]).ToLocalChecked());
   {%elsif cppClassName|isV8Value %}
 
     {%if cType|isPointer %}
@@ -69,7 +69,7 @@
   {%elsif cppClassName == 'GitOid'%}
   if (info[{{ jsArg }}]->IsString()) {
     // Try and parse in a string to a git_oid
-    String::Utf8Value oidString(info[{{ jsArg }}]->ToString());
+   Nan::Utf8String oidString(Nan::To<v8::String>(info[{{ jsArg }}]).ToLocalChecked());
     git_oid *oidOut = (git_oid *)malloc(sizeof(git_oid));
 
     if (git_oid_fromstr(oidOut, (const char *) strdup(*oidString)) != GIT_OK) {
@@ -89,10 +89,10 @@
     {%endif%}
   }
   else {
-    {%if cType|isDoublePointer %}*{%endif%}from_{{ name }} = Nan::ObjectWrap::Unwrap<{{ cppClassName }}>(info[{{ jsArg }}]->ToObject())->GetValue();
+    {%if cType|isDoublePointer %}*{%endif%}from_{{ name }} = Nan::ObjectWrap::Unwrap<{{ cppClassName }}>(Nan::To<v8::Object>(info[{{ jsArg }}]).ToLocalChecked())->GetValue();
   }
   {%else%}
-    {%if cType|isDoublePointer %}*{%endif%}from_{{ name }} = Nan::ObjectWrap::Unwrap<{{ cppClassName }}>(info[{{ jsArg }}]->ToObject())->GetValue();
+    {%if cType|isDoublePointer %}*{%endif%}from_{{ name }} = Nan::ObjectWrap::Unwrap<{{ cppClassName }}>(Nan::To<v8::Object>(info[{{ jsArg }}]).ToLocalChecked())->GetValue();
   {%endif%}
 
   {%if isBoolean %}
diff --git a/generate/templates/partials/convert_to_v8.cc b/generate/templates/partials/convert_to_v8.cc
index 1227932a6..a7a251938 100644
--- a/generate/templates/partials/convert_to_v8.cc
+++ b/generate/templates/partials/convert_to_v8.cc
@@ -79,11 +79,11 @@
         {% if isAsync %}
           {% each ownedBy as owner %}
             {%-- If the owner of this object is "this" in an async method, it will be stored in the persistent handle by name. --%}
-            Nan::Set(owners, Nan::New<v8::Number>(owners->Length()), this->GetFromPersistent("{{= owner =}}")->ToObject());
+            Nan::Set(owners, Nan::New<v8::Number>(owners->Length()), Nan::To<v8::Object>(this->GetFromPersistent("{{= owner =}}")).ToLocalChecked());
           {% endeach %}
         {% else %}
           {% each ownedByIndices as ownedByIndex %}
-            Nan::Set(owners, Nan::New<v8::Number>(owners->Length()), info[{{= ownedByIndex =}}]->ToObject());
+            Nan::Set(owners, Nan::New<v8::Number>(owners->Length()), Nan::To<v8::Object>(info[{{= ownedByIndex =}}]).ToLocalChecked());
           {% endeach %}
         {% endif %}
       {% endif %}
@@ -96,10 +96,10 @@
         Nan::Set(
           owners,
           Nan::New<v8::Number>(owners->Length()),
-          {{= ownerFn.singletonCppClassName =}}::New(
+          Nan::To<v8::Object>({{= ownerFn.singletonCppClassName =}}::New(
             {{= ownerFn.name =}}({{ cType|asElementPointer parsedName }}),
             true
-          )->ToObject()
+          )).ToLocalChecked()
         );
       {% endif %}
     {% endif %}
diff --git a/generate/templates/partials/field_accessors.cc b/generate/templates/partials/field_accessors.cc
index 222034150..8ed8858fa 100644
--- a/generate/templates/partials/field_accessors.cc
+++ b/generate/templates/partials/field_accessors.cc
@@ -39,11 +39,11 @@
         }
 
       {% elsif field.isLibgitType %}
-        v8::Local<Object> {{ field.name }}(value->ToObject());
+        v8::Local<Object> {{ field.name }}(Nan::To<v8::Object>(value).ToLocalChecked());
 
         wrapper->{{ field.name }}.Reset({{ field.name }});
 
-        wrapper->raw->{{ field.name }} = {% if not field.cType | isPointer %}*{% endif %}{% if field.cppClassName == 'GitStrarray' %}StrArrayConverter::Convert({{ field.name }}->ToObject()){% else %}Nan::ObjectWrap::Unwrap<{{ field.cppClassName }}>({{ field.name }}->ToObject())->GetValue(){% endif %};
+        wrapper->raw->{{ field.name }} = {% if not field.cType | isPointer %}*{% endif %}{% if field.cppClassName == 'GitStrarray' %}StrArrayConverter::Convert(Nan::To<v8::Object>({{ field.name }}).ToLocalChecked())){% else %}Nan::ObjectWrap::Unwrap<{{ field.cppClassName }}>(Nan::To<v8::Object>({{ field.name }}).ToLocalChecked())->GetValue(){% endif %};
 
       {% elsif field.isCallbackFunction %}
         Nan::Callback *callback = NULL;
@@ -92,7 +92,7 @@
         if (wrapper->GetValue()->{{ field.name }}) {
         }
 
-        String::Utf8Value str(value);
+       Nan::Utf8String str(value);
         wrapper->GetValue()->{{ field.name }} = strdup(*str);
 
       {% elsif field.isCppClassIntType %}
@@ -225,7 +225,7 @@
             }
             else if (!result->IsNull() && !result->IsUndefined()) {
               {% if _return.isOutParam %}
-              {{ _return.cppClassName }}* wrapper = Nan::ObjectWrap::Unwrap<{{ _return.cppClassName }}>(result->ToObject());
+              {{ _return.cppClassName }}* wrapper = Nan::ObjectWrap::Unwrap<{{ _return.cppClassName }}>(Nan::To<v8::Object>(result).ToLocalChecked());
               wrapper->selfFreeing = false;
 
               *baton->{{ _return.name }} = wrapper->GetValue();
@@ -261,7 +261,7 @@
               }
               else if (!result->IsNull() && !result->IsUndefined()) {
                 {% if _return.isOutParam %}
-                {{ _return.cppClassName }}* wrapper = Nan::ObjectWrap::Unwrap<{{ _return.cppClassName }}>(result->ToObject());
+                {{ _return.cppClassName }}* wrapper = Nan::ObjectWrap::Unwrap<{{ _return.cppClassName }}>(Nan::To<v8::Object>(result).ToLocalChecked());
                 wrapper->selfFreeing = false;
 
                 *baton->{{ _return.name }} = wrapper->GetValue();
diff --git a/generate/templates/partials/sync_function.cc b/generate/templates/partials/sync_function.cc
index 89bb2e2dc..4a76463ed 100644
--- a/generate/templates/partials/sync_function.cc
+++ b/generate/templates/partials/sync_function.cc
@@ -17,7 +17,7 @@ NAN_METHOD({{ cppClassName }}::{{ cppFunctionName }}) {
       {%if not arg.isReturn %}
         {%partial convertFromV8 arg %}
         {%if arg.saveArg %}
-          v8::Local<Object> {{ arg.name }}(info[{{ arg.jsArg }}]->ToObject());
+          v8::Local<Object> {{ arg.name }}(Nan::To<v8::Object>(info[{{ arg.jsArg }}]).ToLocalChecked());
           {{ cppClassName }} *thisObj = Nan::ObjectWrap::Unwrap<{{ cppClassName }}>(info.This());
 
           thisObj->{{ cppFunctionName }}_{{ arg.name }}.Reset({{ arg.name }});
diff --git a/generate/templates/templates/struct_content.cc b/generate/templates/templates/struct_content.cc
index 0c7b9e4f1..0450447ac 100644
--- a/generate/templates/templates/struct_content.cc
+++ b/generate/templates/templates/struct_content.cc
@@ -80,10 +80,10 @@ void {{ cppClassName }}::ConstructFields() {
     {% if not field.ignore %}
       {% if not field.isEnum %}
         {% if field.hasConstructor |or field.isLibgitType %}
-          v8::Local<Object> {{ field.name }}Temp = {{ field.cppClassName }}::New(
+          v8::Local<Object> {{ field.name }}Temp = Nan::To<v8::Object>({{ field.cppClassName }}::New(
             {%if not field.cType|isPointer %}&{%endif%}this->raw->{{ field.name }},
             false
-          )->ToObject();
+          )).ToLocalChecked();
           this->{{ field.name }}.Reset({{ field.name }}Temp);
 
         {% elsif field.isCallbackFunction %}

From 59bad990829711ee4e499ed27892ecb6cef5900a Mon Sep 17 00:00:00 2001
From: Peter Hunt <peter.hunt@bglgroup.co.uk>
Date: Wed, 10 Jul 2019 09:52:45 +0100
Subject: [PATCH 02/13] :arrow_up: GetPropertyNames also needs to be changed
 for Node 12

---
 generate/templates/manual/clone/clone.cc       | 2 +-
 generate/templates/manual/filter_list/load.cc  | 4 ++--
 generate/templates/manual/revwalk/fast_walk.cc | 2 +-
 generate/templates/partials/async_function.cc  | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/generate/templates/manual/clone/clone.cc b/generate/templates/manual/clone/clone.cc
index 2cab928bf..19c44af21 100644
--- a/generate/templates/manual/clone/clone.cc
+++ b/generate/templates/manual/clone/clone.cc
@@ -180,7 +180,7 @@ void GitClone::CloneWorker::HandleOKCallback() {
           break;
         }
 
-        v8::Local<v8::Array> properties = nodeObj->GetPropertyNames();
+        v8::Local<v8::Array> properties = Nan::GetPropertyNames(nodeObj).ToLocalChecked();
         for (unsigned int propIndex = 0; propIndex < properties->Length();
              ++propIndex) {
           v8::Local<v8::String> propName =
diff --git a/generate/templates/manual/filter_list/load.cc b/generate/templates/manual/filter_list/load.cc
index 9a3e1ba25..9075b5ead 100644
--- a/generate/templates/manual/filter_list/load.cc
+++ b/generate/templates/manual/filter_list/load.cc
@@ -134,7 +134,7 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
       // GitFilterList baton->filters
       v8::Local<v8::Array> owners = Nan::New<Array>(0);
       v8::Local<v8::Object> filterRegistry = Nan::New(GitFilterRegistry::persistentHandle);
-      v8::Local<v8::Array> propertyNames = filterRegistry->GetPropertyNames();
+      v8::Local<v8::Array> propertyNames = Nan::GetPropertyNames(filterRegistry).ToLocalChecked();
 
       Nan::Set(
         owners,
@@ -214,7 +214,7 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
           break;
         }
 
-        v8::Local<v8::Array> properties = nodeObj->GetPropertyNames();
+        v8::Local<v8::Array> properties = Nan::GetPropertyNames(nodeObj).ToLocalChecked();
         for (unsigned int propIndex = 0; propIndex < properties->Length();
              ++propIndex) {
           v8::Local<v8::String> propName =
diff --git a/generate/templates/manual/revwalk/fast_walk.cc b/generate/templates/manual/revwalk/fast_walk.cc
index 619f8c6c8..bf4f70710 100644
--- a/generate/templates/manual/revwalk/fast_walk.cc
+++ b/generate/templates/manual/revwalk/fast_walk.cc
@@ -144,7 +144,7 @@ void GitRevwalk::FastWalkWorker::HandleOKCallback()
           break;
         }
 
-        Local<v8::Array> properties = nodeObj->GetPropertyNames();
+        Local<v8::Array> properties = Nan::GetPropertyNames(nodeObj).ToLocalChecked();
         for (unsigned int propIndex = 0; propIndex < properties->Length(); ++propIndex)
         {
           Local<v8::String> propName = Nan::To<v8::String>(properties->Get(propIndex)).ToLocalChecked();
diff --git a/generate/templates/partials/async_function.cc b/generate/templates/partials/async_function.cc
index ea7c43037..8cd11d0f9 100644
--- a/generate/templates/partials/async_function.cc
+++ b/generate/templates/partials/async_function.cc
@@ -217,7 +217,7 @@ void {{ cppClassName }}::{{ cppFunctionName }}Worker::HandleOKCallback() {
           break;
         }
 
-        v8::Local<v8::Array> properties = nodeObj->GetPropertyNames();
+        v8::Local<v8::Array> properties = Nan::GetPropertyNames(nodeObj).ToLocalChecked();
         for (unsigned int propIndex = 0; propIndex < properties->Length(); ++propIndex) {
           v8::Local<v8::String> propName = Nan::To<v8::String>(properties->Get(propIndex)).ToLocalChecked();
           v8::Local<v8::Value> nodeToQueue = nodeObj->Get(propName);

From eb2df373ec490679b2b9aa69e26b5d85f56feadf Mon Sep 17 00:00:00 2001
From: Peter Hunt <peter.hunt@bglgroup.co.uk>
Date: Wed, 10 Jul 2019 09:56:59 +0100
Subject: [PATCH 03/13] :art: Extra space for Utf8Strings to line up better

---
 generate/templates/manual/clone/clone.cc               | 4 ++--
 generate/templates/manual/commit/extract_signature.cc  | 4 ++--
 generate/templates/manual/filter_list/load.cc          | 4 ++--
 generate/templates/manual/revwalk/file_history_walk.cc | 2 +-
 generate/templates/manual/src/filter_registry.cc       | 4 ++--
 generate/templates/manual/src/str_array_converter.cc   | 4 ++--
 generate/templates/partials/convert_from_v8.cc         | 6 +++---
 generate/templates/partials/field_accessors.cc         | 2 +-
 8 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/generate/templates/manual/clone/clone.cc b/generate/templates/manual/clone/clone.cc
index 19c44af21..e0921e700 100644
--- a/generate/templates/manual/clone/clone.cc
+++ b/generate/templates/manual/clone/clone.cc
@@ -33,7 +33,7 @@ NAN_METHOD(GitClone::Clone) {
   // start convert_from_v8 block
   const char *from_url = NULL;
 
- Nan::Utf8String url(Nan::To<v8::String>(info[0]).ToLocalChecked());
+  Nan::Utf8String url(Nan::To<v8::String>(info[0]).ToLocalChecked());
   // malloc with one extra byte so we can add the terminating null character
   // C-strings expect:
   from_url = (const char *)malloc(url.length() + 1);
@@ -50,7 +50,7 @@ NAN_METHOD(GitClone::Clone) {
   // start convert_from_v8 block
   const char *from_local_path = NULL;
 
- Nan::Utf8String local_path(Nan::To<v8::String>(info[1]).ToLocalChecked());
+  Nan::Utf8String local_path(Nan::To<v8::String>(info[1]).ToLocalChecked());
   // malloc with one extra byte so we can add the terminating null character
   // C-strings expect:
   from_local_path = (const char *)malloc(local_path.length() + 1);
diff --git a/generate/templates/manual/commit/extract_signature.cc b/generate/templates/manual/commit/extract_signature.cc
index ebcb78fe0..a5bd9f8b1 100644
--- a/generate/templates/manual/commit/extract_signature.cc
+++ b/generate/templates/manual/commit/extract_signature.cc
@@ -32,7 +32,7 @@ NAN_METHOD(GitCommit::ExtractSignature)
 
   // baton->commit_id
   if (info[1]->IsString()) {
-   Nan::Utf8String oidString(Nan::To<v8::String>(info[1]).ToLocalChecked());
+    Nan::Utf8String oidString(Nan::To<v8::String>(info[1]).ToLocalChecked());
     baton->commit_id = (git_oid *)malloc(sizeof(git_oid));
     if (git_oid_fromstr(baton->commit_id, (const char *)strdup(*oidString)) != GIT_OK) {
       free(baton->commit_id);
@@ -49,7 +49,7 @@ NAN_METHOD(GitCommit::ExtractSignature)
 
   // baton->field
   if (info[2]->IsString()) {
-   Nan::Utf8String field(Nan::To<v8::String>(info[2]).ToLocalChecked());
+    Nan::Utf8String field(Nan::To<v8::String>(info[2]).ToLocalChecked());
     baton->field = (char *)malloc(field.length() + 1);
     memcpy((void *)baton->field, *field, field.length());
     baton->field[field.length()] = 0;
diff --git a/generate/templates/manual/filter_list/load.cc b/generate/templates/manual/filter_list/load.cc
index 9075b5ead..082cfe212 100644
--- a/generate/templates/manual/filter_list/load.cc
+++ b/generate/templates/manual/filter_list/load.cc
@@ -63,7 +63,7 @@ NAN_METHOD(GitFilterList::Load) {
   // start convert_from_v8 block
   const char *from_path = NULL;
 
- Nan::Utf8String path(Nan::To<v8::String>(info[2]).ToLocalChecked());
+  Nan::Utf8String path(Nan::To<v8::String>(info[2]).ToLocalChecked());
   // malloc with one extra byte so we can add the terminating null character
   // C-strings expect:
   from_path = (const char *)malloc(path.length() + 1);
@@ -144,7 +144,7 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
 
       for (uint32_t index = 0; index < propertyNames->Length(); ++index) {
         v8::Local<v8::String> propertyName = Nan::To<v8::String>(propertyNames->Get(index)).ToLocalChecked();
-       Nan::Utf8String propertyNameAsUtf8Value(propertyName);
+        Nan::Utf8String propertyNameAsUtf8Value(propertyName);
         const char *propertyNameAsCString = *propertyNameAsUtf8Value;
 
         bool isNotMethodOnRegistry = strcmp("register", propertyNameAsCString)
diff --git a/generate/templates/manual/revwalk/file_history_walk.cc b/generate/templates/manual/revwalk/file_history_walk.cc
index f7cec692a..a3d9764d6 100644
--- a/generate/templates/manual/revwalk/file_history_walk.cc
+++ b/generate/templates/manual/revwalk/file_history_walk.cc
@@ -196,7 +196,7 @@ NAN_METHOD(GitRevwalk::FileHistoryWalk)
 
   baton->error_code = GIT_OK;
   baton->error = NULL;
- Nan::Utf8String from_js_file_path(Nan::To<v8::String>(info[0]).ToLocalChecked());
+  Nan::Utf8String from_js_file_path(Nan::To<v8::String>(info[0]).ToLocalChecked());
   baton->file_path = strdup(*from_js_file_path);
   baton->max_count = Nan::To<unsigned int>(info[1]).FromJust();
   baton->out = new std::vector<void *>;
diff --git a/generate/templates/manual/src/filter_registry.cc b/generate/templates/manual/src/filter_registry.cc
index 62de8f80a..bf69012d4 100644
--- a/generate/templates/manual/src/filter_registry.cc
+++ b/generate/templates/manual/src/filter_registry.cc
@@ -55,7 +55,7 @@ NAN_METHOD(GitFilterRegistry::GitFilterRegister) {
   FilterRegisterBaton *baton = new FilterRegisterBaton;
 
   baton->filter = Nan::ObjectWrap::Unwrap<GitFilter>(Nan::To<v8::Object>(info[1]).ToLocalChecked())->GetValue();
- Nan::Utf8String name(Nan::To<v8::String>(info[0]).ToLocalChecked());
+  Nan::Utf8String name(Nan::To<v8::String>(info[0]).ToLocalChecked());
 
   baton->filter_name = (char *)malloc(name.length() + 1);
   memcpy((void *)baton->filter_name, *name, name.length());
@@ -144,7 +144,7 @@ NAN_METHOD(GitFilterRegistry::GitFilterUnregister) {
   }
 
   FilterUnregisterBaton *baton = new FilterUnregisterBaton;
- Nan::Utf8String name(Nan::To<v8::String>(info[0]).ToLocalChecked());
+  Nan::Utf8String name(Nan::To<v8::String>(info[0]).ToLocalChecked());
 
   baton->filter_name = (char *)malloc(name.length() + 1);
   memcpy((void *)baton->filter_name, *name, name.length());
diff --git a/generate/templates/manual/src/str_array_converter.cc b/generate/templates/manual/src/str_array_converter.cc
index ed0e93e2b..56e7d7dda 100644
--- a/generate/templates/manual/src/str_array_converter.cc
+++ b/generate/templates/manual/src/str_array_converter.cc
@@ -37,7 +37,7 @@ git_strarray *StrArrayConverter::ConvertArray(Array *val) {
   git_strarray *result = AllocStrArray(val->Length());
 
   for(size_t i = 0; i < result->count; i++) {
-    Nan::Utf8String entry(val->Get(i));
+     Nan::Utf8String entry(val->Get(i));
     result->strings[i] = strdup(*entry);
   }
 
@@ -46,7 +46,7 @@ git_strarray *StrArrayConverter::ConvertArray(Array *val) {
 
 git_strarray* StrArrayConverter::ConvertString(Local<String> val) {
   char *strings[1];
-  Nan::Utf8String utf8String(val);
+   Nan::Utf8String utf8String(val);
 
   strings[0] = *utf8String;
 
diff --git a/generate/templates/partials/convert_from_v8.cc b/generate/templates/partials/convert_from_v8.cc
index 4f6a146d4..a3ea193a5 100644
--- a/generate/templates/partials/convert_from_v8.cc
+++ b/generate/templates/partials/convert_from_v8.cc
@@ -19,7 +19,7 @@
     {%endif%}
   {%if cppClassName == 'String'%}
 
- Nan::Utf8String {{ name }}(Nan::To<v8::String>(info[{{ jsArg }}]).ToLocalChecked());
+  Nan::Utf8String {{ name }}(Nan::To<v8::String>(info[{{ jsArg }}]).ToLocalChecked());
   // malloc with one extra byte so we can add the terminating null character C-strings expect:
   from_{{ name }} = ({{ cType }}) malloc({{ name }}.length() + 1);
   // copy the characters from the nodejs string into our C-string (used instead of strdup or strcpy because nulls in
@@ -36,7 +36,7 @@
   from_{{ name }} = GitBufConverter::Convert(info[{{ jsArg }}]);
   {%elsif cppClassName == 'Wrapper'%}
 
- Nan::Utf8String {{ name }}(Nan::To<v8::String>(info[{{ jsArg }}]).ToLocalChecked());
+  Nan::Utf8String {{ name }}(Nan::To<v8::String>(info[{{ jsArg }}]).ToLocalChecked());
   // malloc with one extra byte so we can add the terminating null character C-strings expect:
   from_{{ name }} = ({{ cType }}) malloc({{ name }}.length() + 1);
   // copy the characters from the nodejs string into our C-string (used instead of strdup or strcpy because nulls in
@@ -69,7 +69,7 @@
   {%elsif cppClassName == 'GitOid'%}
   if (info[{{ jsArg }}]->IsString()) {
     // Try and parse in a string to a git_oid
-   Nan::Utf8String oidString(Nan::To<v8::String>(info[{{ jsArg }}]).ToLocalChecked());
+    Nan::Utf8String oidString(Nan::To<v8::String>(info[{{ jsArg }}]).ToLocalChecked());
     git_oid *oidOut = (git_oid *)malloc(sizeof(git_oid));
 
     if (git_oid_fromstr(oidOut, (const char *) strdup(*oidString)) != GIT_OK) {
diff --git a/generate/templates/partials/field_accessors.cc b/generate/templates/partials/field_accessors.cc
index 8ed8858fa..c805d8d13 100644
--- a/generate/templates/partials/field_accessors.cc
+++ b/generate/templates/partials/field_accessors.cc
@@ -92,7 +92,7 @@
         if (wrapper->GetValue()->{{ field.name }}) {
         }
 
-       Nan::Utf8String str(value);
+        Nan::Utf8String str(value);
         wrapper->GetValue()->{{ field.name }} = strdup(*str);
 
       {% elsif field.isCppClassIntType %}

From 8e723ef46635b912c5d47a3b389f075a58547fe4 Mon Sep 17 00:00:00 2001
From: Peter Hunt <peter.hunt@bglgroup.co.uk>
Date: Wed, 10 Jul 2019 10:03:55 +0100
Subject: [PATCH 04/13] :arrow_up: BooleanValue also needs changing for Node 12

---
 generate/templates/manual/src/str_array_converter.cc | 2 +-
 generate/templates/partials/field_accessors.cc       | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/generate/templates/manual/src/str_array_converter.cc b/generate/templates/manual/src/str_array_converter.cc
index 56e7d7dda..afca4bd06 100644
--- a/generate/templates/manual/src/str_array_converter.cc
+++ b/generate/templates/manual/src/str_array_converter.cc
@@ -10,7 +10,7 @@ using namespace v8;
 using namespace node;
 
 git_strarray *StrArrayConverter::Convert(Local<v8::Value> val) {
-  if (!val->BooleanValue()) {
+  if (!Nan::To<bool>(val).FromJust()) {
     return NULL;
   }
   else if (val->IsArray()) {
diff --git a/generate/templates/partials/field_accessors.cc b/generate/templates/partials/field_accessors.cc
index c805d8d13..fd785828c 100644
--- a/generate/templates/partials/field_accessors.cc
+++ b/generate/templates/partials/field_accessors.cc
@@ -72,7 +72,7 @@
               Nan::MaybeLocal<Value> maybeObjectWaitForResult = Nan::Get(object, Nan::New("waitForResult").ToLocalChecked());
               if(!maybeObjectWaitForResult.IsEmpty()) {
                 Local<Value> objectWaitForResult = maybeObjectWaitForResult.ToLocalChecked();
-                waitForResult = (bool)objectWaitForResult->BooleanValue();
+                waitForResult = Nan::To<bool>(objectWaitForResult).FromJust();
               }
             }
           }

From 77b615a6ebd0f910a816b76142eabea830dc9a2e Mon Sep 17 00:00:00 2001
From: Peter Hunt <peter.hunt@bglgroup.co.uk>
Date: Wed, 10 Jul 2019 10:10:06 +0100
Subject: [PATCH 05/13] :arrow_up: Minor amends for a few manual templates

---
 generate/templates/manual/src/filter_registry.cc   | 2 +-
 generate/templates/manual/src/git_buf_converter.cc | 2 +-
 generate/templates/manual/src/nodegit_wrapper.cc   | 2 +-
 generate/templates/partials/field_accessors.cc     | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/generate/templates/manual/src/filter_registry.cc b/generate/templates/manual/src/filter_registry.cc
index bf69012d4..59124ab41 100644
--- a/generate/templates/manual/src/filter_registry.cc
+++ b/generate/templates/manual/src/filter_registry.cc
@@ -64,7 +64,7 @@ NAN_METHOD(GitFilterRegistry::GitFilterRegister) {
   baton->error_code = GIT_OK;
   baton->filter_priority = Nan::To<int>(info[2]).FromJust();
 
-  Nan::New(GitFilterRegistry::persistentHandle)->Set(Nan::To<v8::Object>(Nan::To<v8::String>(info[0]).ToLocalChecked(), info[1]).ToLocalChecked());
+  Nan::New(GitFilterRegistry::persistentHandle)->Set(Nan::To<v8::String>(info[0]).ToLocalChecked(), Nan::To<v8::Object>(info[1]).ToLocalChecked());
 
   Nan::Callback *callback = new Nan::Callback(Local<Function>::Cast(info[3]));
   RegisterWorker *worker = new RegisterWorker(baton, callback);
diff --git a/generate/templates/manual/src/git_buf_converter.cc b/generate/templates/manual/src/git_buf_converter.cc
index 1413168af..1558d39fe 100644
--- a/generate/templates/manual/src/git_buf_converter.cc
+++ b/generate/templates/manual/src/git_buf_converter.cc
@@ -10,7 +10,7 @@ using namespace node;
 
 git_buf *GitBufConverter::Convert(Local<v8::Value> val) {
   if (val->IsString() || val->IsStringObject()) {
-    v8::String::Utf8Value param1(Nan::To<v8::String>(val).ToLocalChecked());
+    Nan::Utf8String param1(Nan::To<v8::String>(val).ToLocalChecked());
     std::string v8String = std::string(*param1);
 
     const size_t size = sizeof(git_buf);
diff --git a/generate/templates/manual/src/nodegit_wrapper.cc b/generate/templates/manual/src/nodegit_wrapper.cc
index e9fb8ebdb..69aed4f94 100644
--- a/generate/templates/manual/src/nodegit_wrapper.cc
+++ b/generate/templates/manual/src/nodegit_wrapper.cc
@@ -67,7 +67,7 @@ NAN_METHOD(NodeGitWrapper<Traits>::JSNewFunction) {
     instance = new cppClass(static_cast<cType *>(
       Local<External>::Cast(info[0])->Value()),
       Nan::To<bool>(info[1]).FromJust(),
-      info.Length() >= 3 && !Nan::To<v8::Object>(info[2].IsEmpty() && info[2]->IsObject() ? info[2]).ToLocalChecked() : Local<v8::Object>()
+      info.Length() >= 3 && !info[2].IsEmpty() && info[2]->IsObject() ? Nan::To<v8::Object>(info[2]).ToLocalChecked() : Local<v8::Object>()
     );
   }
 
diff --git a/generate/templates/partials/field_accessors.cc b/generate/templates/partials/field_accessors.cc
index fd785828c..35b583bdc 100644
--- a/generate/templates/partials/field_accessors.cc
+++ b/generate/templates/partials/field_accessors.cc
@@ -43,7 +43,7 @@
 
         wrapper->{{ field.name }}.Reset({{ field.name }});
 
-        wrapper->raw->{{ field.name }} = {% if not field.cType | isPointer %}*{% endif %}{% if field.cppClassName == 'GitStrarray' %}StrArrayConverter::Convert(Nan::To<v8::Object>({{ field.name }}).ToLocalChecked())){% else %}Nan::ObjectWrap::Unwrap<{{ field.cppClassName }}>(Nan::To<v8::Object>({{ field.name }}).ToLocalChecked())->GetValue(){% endif %};
+        wrapper->raw->{{ field.name }} = {% if not field.cType | isPointer %}*{% endif %}{% if field.cppClassName == 'GitStrarray' %}StrArrayConverter::Convert(Nan::To<v8::Object>({{ field.name }}).ToLocalChecked()){% else %}Nan::ObjectWrap::Unwrap<{{ field.cppClassName }}>(Nan::To<v8::Object>({{ field.name }}).ToLocalChecked())->GetValue(){% endif %};
 
       {% elsif field.isCallbackFunction %}
         Nan::Callback *callback = NULL;

From a470a678b4204f28b4e4de607a103ae2b254b5a1 Mon Sep 17 00:00:00 2001
From: Peter Hunt <peter.hunt@bglgroup.co.uk>
Date: Wed, 10 Jul 2019 10:16:26 +0100
Subject: [PATCH 06/13] :arrow_up: Nan::Call requires ToLocalChecked

---
 generate/templates/manual/src/promise_completion.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/generate/templates/manual/src/promise_completion.cc b/generate/templates/manual/src/promise_completion.cc
index 88bde47da..1e503ac72 100644
--- a/generate/templates/manual/src/promise_completion.cc
+++ b/generate/templates/manual/src/promise_completion.cc
@@ -78,7 +78,7 @@ v8::Local<v8::Value> PromiseCompletion::Bind(Nan::Persistent<v8::Function> &func
 
   v8::Local<v8::Value> argv[1] = { object };
 
-  return scope.Escape(Nan::Call(bind, Nan::To<v8::Object>(Nan::New(function)).ToLocalChecked(), 1, argv));
+  return scope.Escape(Nan::Call(bind, Nan::To<v8::Object>(Nan::New(function)).ToLocalChecked(), 1, argv).ToLocalChecked());
 }
 
 // calls the callback stored in the PromiseCompletion, passing the baton that

From a10a61677a479d5e83b9d7b4ff962f2dd8439098 Mon Sep 17 00:00:00 2001
From: Peter Hunt <peter.hunt@bglgroup.co.uk>
Date: Wed, 10 Jul 2019 10:25:15 +0100
Subject: [PATCH 07/13] :art: Final spacing amends for manual
 str_array_converter

---
 generate/templates/manual/src/str_array_converter.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/generate/templates/manual/src/str_array_converter.cc b/generate/templates/manual/src/str_array_converter.cc
index afca4bd06..6100fd01d 100644
--- a/generate/templates/manual/src/str_array_converter.cc
+++ b/generate/templates/manual/src/str_array_converter.cc
@@ -37,7 +37,7 @@ git_strarray *StrArrayConverter::ConvertArray(Array *val) {
   git_strarray *result = AllocStrArray(val->Length());
 
   for(size_t i = 0; i < result->count; i++) {
-     Nan::Utf8String entry(val->Get(i));
+    Nan::Utf8String entry(val->Get(i));
     result->strings[i] = strdup(*entry);
   }
 
@@ -46,7 +46,7 @@ git_strarray *StrArrayConverter::ConvertArray(Array *val) {
 
 git_strarray* StrArrayConverter::ConvertString(Local<String> val) {
   char *strings[1];
-   Nan::Utf8String utf8String(val);
+  Nan::Utf8String utf8String(val);
 
   strings[0] = *utf8String;
 

From 4db16704391f058e11c8fd7cab00a7cfc315200f Mon Sep 17 00:00:00 2001
From: Peter Hunt <peter.hunt@bglgroup.co.uk>
Date: Fri, 12 Jul 2019 08:29:02 +0100
Subject: [PATCH 08/13] :white_check_mark: Start building for Node 12

---
 .travis.yml  | 1 +
 appveyor.yml | 1 +
 2 files changed, 2 insertions(+)

diff --git a/.travis.yml b/.travis.yml
index 682ca1cf9..e1d7ff047 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -20,6 +20,7 @@ env:
   - TARGET_ARCH="ia32"
 
 node_js:
+  - "12"
   - "10"
   - "8"
 
diff --git a/appveyor.yml b/appveyor.yml
index 46fc6a1b5..891225fd7 100644
--- a/appveyor.yml
+++ b/appveyor.yml
@@ -26,6 +26,7 @@ environment:
   GYP_MSVS_VERSION: 2015
   matrix:
     # Node.js
+    - nodejs_version: "12"
     - nodejs_version: "10"
     - nodejs_version: "8"
 

From 9bc65c6d242bb0e3cd1350408925c79c8426bdd7 Mon Sep 17 00:00:00 2001
From: Peter Hunt <peter.hunt@bglgroup.co.uk>
Date: Fri, 12 Jul 2019 09:12:50 +0100
Subject: [PATCH 09/13] :arrow_up: Update versions of nan and node-pre-gyp

---
 package.json | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package.json b/package.json
index a2ab166b5..c071d62c7 100644
--- a/package.json
+++ b/package.json
@@ -41,9 +41,9 @@
     "fs-extra": "^7.0.0",
     "json5": "^2.1.0",
     "lodash": "^4.17.11",
-    "nan": "^2.11.1",
+    "nan": "^2.14.0",
     "node-gyp": "^4.0.0",
-    "node-pre-gyp": "^0.11.0",
+    "node-pre-gyp": "^0.13.0",
     "promisify-node": "~0.3.0",
     "ramda": "^0.25.0",
     "request-promise-native": "^1.0.5",

From 658936e8faa97eb555ea58f21c888e1925299861 Mon Sep 17 00:00:00 2001
From: Peter Hunt <peter.hunt@bglgroup.co.uk>
Date: Fri, 12 Jul 2019 11:48:46 +0100
Subject: [PATCH 10/13] :recycle: Remove last depreciated warnings so
 everything is now clean

---
 generate/templates/manual/clone/clone.cc      | 12 +++++-----
 .../manual/commit/extract_signature.cc        |  4 ++--
 generate/templates/manual/filter_list/load.cc | 16 ++++++-------
 .../templates/manual/filter_source/repo.cc    |  8 +++----
 .../manual/include/str_array_converter.h      |  4 ++--
 .../manual/patches/convenient_patches.cc      |  8 +++----
 generate/templates/manual/remote/ls.cc        |  4 ++--
 .../manual/repository/get_references.cc       |  4 ++--
 .../manual/repository/get_remotes.cc          |  4 ++--
 .../manual/repository/get_submodules.cc       |  4 ++--
 .../manual/repository/refresh_references.cc   |  4 ++--
 .../templates/manual/revwalk/commit_walk.cc   |  4 ++--
 .../templates/manual/revwalk/fast_walk.cc     | 12 +++++-----
 .../manual/revwalk/file_history_walk.cc       |  8 +++----
 .../templates/manual/src/filter_registry.cc   | 18 +++++++-------
 .../manual/src/promise_completion.cc          |  2 +-
 .../manual/src/str_array_converter.cc         | 10 ++++----
 generate/templates/partials/async_function.cc | 12 +++++-----
 .../templates/partials/convert_from_v8.cc     |  4 ++--
 generate/templates/templates/nodegit.cc       | 24 +++++++------------
 20 files changed, 80 insertions(+), 86 deletions(-)

diff --git a/generate/templates/manual/clone/clone.cc b/generate/templates/manual/clone/clone.cc
index e0921e700..6f5f79769 100644
--- a/generate/templates/manual/clone/clone.cc
+++ b/generate/templates/manual/clone/clone.cc
@@ -144,8 +144,8 @@ void GitClone::CloneWorker::HandleOKCallback() {
       } else {
         err = Nan::To<v8::Object>(Nan::Error("Method clone has thrown an error.")).ToLocalChecked();
       }
-      err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-      err->Set(Nan::New("errorFunction").ToLocalChecked(),
+      Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+      Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(),
                Nan::New("Clone.clone").ToLocalChecked());
       v8::Local<v8::Value> argv[1] = {err};
       callback->Call(1, argv, async_resource);
@@ -184,8 +184,8 @@ void GitClone::CloneWorker::HandleOKCallback() {
         for (unsigned int propIndex = 0; propIndex < properties->Length();
              ++propIndex) {
           v8::Local<v8::String> propName =
-              Nan::To<v8::String>(properties->Get(propIndex)).ToLocalChecked();
-          v8::Local<v8::Value> nodeToQueue = nodeObj->Get(propName);
+              Nan::To<v8::String>(Nan::Get(properties, propIndex).ToLocalChecked()).ToLocalChecked();
+          v8::Local<v8::Value> nodeToQueue = Nan::Get(nodeObj, propName).ToLocalChecked();
           if (!nodeToQueue->IsUndefined()) {
             workerArguments.push(nodeToQueue);
           }
@@ -195,9 +195,9 @@ void GitClone::CloneWorker::HandleOKCallback() {
       if (!callbackFired) {
         v8::Local<v8::Object> err =
             Nan::To<v8::Object>(Nan::Error("Method clone has thrown an error.")).ToLocalChecked();
-        err->Set(Nan::New("errno").ToLocalChecked(),
+        Nan::Set(err,Nan::New("errno").ToLocalChecked(),
                  Nan::New(baton->error_code));
-        err->Set(Nan::New("errorFunction").ToLocalChecked(),
+        Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(),
                  Nan::New("Clone.clone").ToLocalChecked());
         v8::Local<v8::Value> argv[1] = {err};
         callback->Call(1, argv, async_resource);
diff --git a/generate/templates/manual/commit/extract_signature.cc b/generate/templates/manual/commit/extract_signature.cc
index a5bd9f8b1..0fea1bc0a 100644
--- a/generate/templates/manual/commit/extract_signature.cc
+++ b/generate/templates/manual/commit/extract_signature.cc
@@ -133,8 +133,8 @@ void GitCommit::ExtractSignatureWorker::HandleOKCallback()
   else if (baton->error_code < 0)
   {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Extract Signature has thrown an error.")).ToLocalChecked();
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Commit.extractSignature").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Commit.extractSignature").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/filter_list/load.cc b/generate/templates/manual/filter_list/load.cc
index 082cfe212..40ce79433 100644
--- a/generate/templates/manual/filter_list/load.cc
+++ b/generate/templates/manual/filter_list/load.cc
@@ -143,7 +143,7 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
       );
 
       for (uint32_t index = 0; index < propertyNames->Length(); ++index) {
-        v8::Local<v8::String> propertyName = Nan::To<v8::String>(propertyNames->Get(index)).ToLocalChecked();
+        v8::Local<v8::String> propertyName = Nan::To<v8::String>(Nan::Get(propertyNames, index).ToLocalChecked()).ToLocalChecked();
         Nan::Utf8String propertyNameAsUtf8Value(propertyName);
         const char *propertyNameAsCString = *propertyNameAsUtf8Value;
 
@@ -153,7 +153,7 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
           Nan::Set(
             owners,
             Nan::New<Number>(owners->Length()),
-            filterRegistry->Get(propertyName)
+            Nan::Get(filterRegistry, propertyName).ToLocalChecked()
           );
         }
       }
@@ -176,8 +176,8 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
       } else {
         err = Nan::To<v8::Object>(Nan::Error("Method load has thrown an error.")).ToLocalChecked();
       }
-      err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-      err->Set(Nan::New("errorFunction").ToLocalChecked(),
+      Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+      Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(),
                Nan::New("FilterList.load").ToLocalChecked());
       v8::Local<v8::Value> argv[1] = {err};
       callback->Call(1, argv, async_resource);
@@ -218,8 +218,8 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
         for (unsigned int propIndex = 0; propIndex < properties->Length();
              ++propIndex) {
           v8::Local<v8::String> propName =
-              Nan::To<v8::String>(properties->Get(propIndex)).ToLocalChecked();
-          v8::Local<v8::Value> nodeToQueue = nodeObj->Get(propName);
+              Nan::To<v8::String>(Nan::Get(properties, propIndex).ToLocalChecked()).ToLocalChecked();
+          v8::Local<v8::Value> nodeToQueue = Nan::Get(nodeObj, propName).ToLocalChecked();
           if (!nodeToQueue->IsUndefined()) {
             workerArguments.push(nodeToQueue);
           }
@@ -229,9 +229,9 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
       if (!callbackFired) {
         v8::Local<v8::Object> err =
             Nan::To<v8::Object>(Nan::Error("Method load has thrown an error.")).ToLocalChecked();
-        err->Set(Nan::New("errno").ToLocalChecked(),
+        Nan::Set(err,Nan::New("errno").ToLocalChecked(),
                  Nan::New(baton->error_code));
-        err->Set(Nan::New("errorFunction").ToLocalChecked(),
+        Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(),
                  Nan::New("FilterList.load").ToLocalChecked());
         v8::Local<v8::Value> argv[1] = {err};
         callback->Call(1, argv, async_resource);
diff --git a/generate/templates/manual/filter_source/repo.cc b/generate/templates/manual/filter_source/repo.cc
index f35fb33a4..fca66a754 100644
--- a/generate/templates/manual/filter_source/repo.cc
+++ b/generate/templates/manual/filter_source/repo.cc
@@ -64,8 +64,8 @@ void GitFilterSource::RepoWorker::HandleOKCallback() {
       } else {
         err = Nan::To<v8::Object>(Nan::Error("Method repo has thrown an error.")).ToLocalChecked();
       }
-      err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-      err->Set(Nan::New("errorFunction").ToLocalChecked(),
+      Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+      Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(),
                Nan::New("FilterSource.repo").ToLocalChecked());
       v8::Local<v8::Value> argv[1] = {err};
       callback->Call(1, argv, async_resource);
@@ -75,9 +75,9 @@ void GitFilterSource::RepoWorker::HandleOKCallback() {
     } else if (baton->error_code < 0) {
       v8::Local<v8::Object> err =
           Nan::To<v8::Object>(Nan::Error("Method repo has thrown an error.")).ToLocalChecked();
-      err->Set(Nan::New("errno").ToLocalChecked(),
+      Nan::Set(err,Nan::New("errno").ToLocalChecked(),
                Nan::New(baton->error_code));
-      err->Set(Nan::New("errorFunction").ToLocalChecked(),
+      Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(),
                Nan::New("FilterSource.repo").ToLocalChecked());
       v8::Local<v8::Value> argv[1] = {err};
       callback->Call(1, argv, async_resource);
diff --git a/generate/templates/manual/include/str_array_converter.h b/generate/templates/manual/include/str_array_converter.h
index 37f1bcc1d..26a82479d 100644
--- a/generate/templates/manual/include/str_array_converter.h
+++ b/generate/templates/manual/include/str_array_converter.h
@@ -14,8 +14,8 @@ class StrArrayConverter {
     static git_strarray *Convert (v8::Local<v8::Value> val);
 
   private:
-    static git_strarray *ConvertArray(Array *val);
-    static git_strarray *ConvertString(v8::Local<String> val);
+    static git_strarray *ConvertArray(v8::Local<v8::Array> val);
+    static git_strarray *ConvertString(v8::Local<v8::String> val);
     static git_strarray *AllocStrArray(const size_t count);
     static git_strarray *ConstructStrArray(int argc, char** argv);
 };
diff --git a/generate/templates/manual/patches/convenient_patches.cc b/generate/templates/manual/patches/convenient_patches.cc
index ef9322a43..b27b1b66e 100644
--- a/generate/templates/manual/patches/convenient_patches.cc
+++ b/generate/templates/manual/patches/convenient_patches.cc
@@ -101,8 +101,8 @@ void GitPatch::ConvenientFromDiffWorker::HandleOKCallback() {
     } else {
       err = Nan::To<v8::Object>(Nan::Error("Method convenientFromDiff has thrown an error.")).ToLocalChecked();
     }
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Patch.convenientFromDiff").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Patch.convenientFromDiff").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
@@ -119,8 +119,8 @@ void GitPatch::ConvenientFromDiffWorker::HandleOKCallback() {
 
   if (baton->error_code < 0) {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("method convenientFromDiff has thrown an error.")).ToLocalChecked();
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Patch.convenientFromDiff").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Patch.convenientFromDiff").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/remote/ls.cc b/generate/templates/manual/remote/ls.cc
index 9fccf4af3..5afcb8840 100644
--- a/generate/templates/manual/remote/ls.cc
+++ b/generate/templates/manual/remote/ls.cc
@@ -86,8 +86,8 @@ void GitRemote::ReferenceListWorker::HandleOKCallback()
   else if (baton->error_code < 0)
   {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Reference List has thrown an error.")).ToLocalChecked();
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Remote.referenceList").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Remote.referenceList").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/repository/get_references.cc b/generate/templates/manual/repository/get_references.cc
index 910352b52..aab66d035 100644
--- a/generate/templates/manual/repository/get_references.cc
+++ b/generate/templates/manual/repository/get_references.cc
@@ -119,8 +119,8 @@ void GitRepository::GetReferencesWorker::HandleOKCallback()
   else if (baton->error_code < 0)
   {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Repository getReferences has thrown an error.")).ToLocalChecked();
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.getReferences").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.getReferences").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/repository/get_remotes.cc b/generate/templates/manual/repository/get_remotes.cc
index 457ea3ebe..cfb4e0b26 100644
--- a/generate/templates/manual/repository/get_remotes.cc
+++ b/generate/templates/manual/repository/get_remotes.cc
@@ -120,8 +120,8 @@ void GitRepository::GetRemotesWorker::HandleOKCallback()
   else if (baton->error_code < 0)
   {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Repository refreshRemotes has thrown an error.")).ToLocalChecked();
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.refreshRemotes").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.refreshRemotes").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/repository/get_submodules.cc b/generate/templates/manual/repository/get_submodules.cc
index 0d73424b1..e97fa6184 100644
--- a/generate/templates/manual/repository/get_submodules.cc
+++ b/generate/templates/manual/repository/get_submodules.cc
@@ -101,8 +101,8 @@ void GitRepository::GetSubmodulesWorker::HandleOKCallback()
   else if (baton->error_code < 0)
   {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Repository getSubmodules has thrown an error.")).ToLocalChecked();
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.getSubmodules").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.getSubmodules").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/repository/refresh_references.cc b/generate/templates/manual/repository/refresh_references.cc
index 8db164fb5..93d85c247 100644
--- a/generate/templates/manual/repository/refresh_references.cc
+++ b/generate/templates/manual/repository/refresh_references.cc
@@ -673,8 +673,8 @@ void GitRepository::RefreshReferencesWorker::HandleOKCallback()
   else if (baton->error_code < 0)
   {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Repository refreshReferences has thrown an error.")).ToLocalChecked();
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.refreshReferences").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.refreshReferences").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/revwalk/commit_walk.cc b/generate/templates/manual/revwalk/commit_walk.cc
index 1c7bed6fc..99f011475 100644
--- a/generate/templates/manual/revwalk/commit_walk.cc
+++ b/generate/templates/manual/revwalk/commit_walk.cc
@@ -111,8 +111,8 @@ void GitRevwalk::CommitWalkWorker::HandleOKCallback() {
     free((void *)baton->error);
   } else if (baton->error_code < 0) {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Revwalk commitWalk has thrown an error.")).ToLocalChecked();
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.commitWalk").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.commitWalk").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/revwalk/fast_walk.cc b/generate/templates/manual/revwalk/fast_walk.cc
index bf4f70710..191bc6b4f 100644
--- a/generate/templates/manual/revwalk/fast_walk.cc
+++ b/generate/templates/manual/revwalk/fast_walk.cc
@@ -94,8 +94,8 @@ void GitRevwalk::FastWalkWorker::HandleOKCallback()
       } else {
         err = Nan::To<v8::Object>(Nan::Error("Method fastWalk has thrown an error.")).ToLocalChecked();
       }
-      err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-      err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fastWalk").ToLocalChecked());
+      Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+      Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fastWalk").ToLocalChecked());
       Local<v8::Value> argv[1] = {
         err
       };
@@ -147,8 +147,8 @@ void GitRevwalk::FastWalkWorker::HandleOKCallback()
         Local<v8::Array> properties = Nan::GetPropertyNames(nodeObj).ToLocalChecked();
         for (unsigned int propIndex = 0; propIndex < properties->Length(); ++propIndex)
         {
-          Local<v8::String> propName = Nan::To<v8::String>(properties->Get(propIndex)).ToLocalChecked();
-          Local<v8::Value> nodeToQueue = nodeObj->Get(propName);
+          Local<v8::String> propName = Nan::To<v8::String>(Nan::Get(properties, propIndex).ToLocalChecked()).ToLocalChecked();
+          Local<v8::Value> nodeToQueue = Nan::Get(nodeObj, propName).ToLocalChecked();
           if (!nodeToQueue->IsUndefined())
           {
             workerArguments.push(nodeToQueue);
@@ -159,8 +159,8 @@ void GitRevwalk::FastWalkWorker::HandleOKCallback()
       if (!callbackFired)
       {
         Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method next has thrown an error.")).ToLocalChecked();
-        err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-        err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fastWalk").ToLocalChecked());
+        Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+        Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fastWalk").ToLocalChecked());
         Local<v8::Value> argv[1] = {
           err
         };
diff --git a/generate/templates/manual/revwalk/file_history_walk.cc b/generate/templates/manual/revwalk/file_history_walk.cc
index a3d9764d6..e55d86e9f 100644
--- a/generate/templates/manual/revwalk/file_history_walk.cc
+++ b/generate/templates/manual/revwalk/file_history_walk.cc
@@ -448,8 +448,8 @@ void GitRevwalk::FileHistoryWalkWorker::HandleOKCallback()
     } else {
       err = Nan::To<v8::Object>(Nan::Error("Method fileHistoryWalk has thrown an error.")).ToLocalChecked();
     }
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fileHistoryWalk").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fileHistoryWalk").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
       err
     };
@@ -465,8 +465,8 @@ void GitRevwalk::FileHistoryWalkWorker::HandleOKCallback()
 
   if (baton->error_code < 0) {
     v8::Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method next has thrown an error.")).ToLocalChecked();
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fileHistoryWalk").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fileHistoryWalk").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/src/filter_registry.cc b/generate/templates/manual/src/filter_registry.cc
index 59124ab41..4014b4abe 100644
--- a/generate/templates/manual/src/filter_registry.cc
+++ b/generate/templates/manual/src/filter_registry.cc
@@ -64,7 +64,7 @@ NAN_METHOD(GitFilterRegistry::GitFilterRegister) {
   baton->error_code = GIT_OK;
   baton->filter_priority = Nan::To<int>(info[2]).FromJust();
 
-  Nan::New(GitFilterRegistry::persistentHandle)->Set(Nan::To<v8::String>(info[0]).ToLocalChecked(), Nan::To<v8::Object>(info[1]).ToLocalChecked());
+  Nan::Set(Nan::New(GitFilterRegistry::persistentHandle), Nan::To<v8::String>(info[0]).ToLocalChecked(), Nan::To<v8::Object>(info[1]).ToLocalChecked());
 
   Nan::Callback *callback = new Nan::Callback(Local<Function>::Cast(info[3]));
   RegisterWorker *worker = new RegisterWorker(baton, callback);
@@ -106,8 +106,8 @@ void GitFilterRegistry::RegisterWorker::HandleOKCallback() {
     } else {
       err = Nan::To<v8::Object>(Nan::Error("Method register has thrown an error.")).ToLocalChecked();
     }
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.register").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.register").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
       err
     };
@@ -118,8 +118,8 @@ void GitFilterRegistry::RegisterWorker::HandleOKCallback() {
   }
   else if (baton->error_code < 0) {
     v8::Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method register has thrown an error.")).ToLocalChecked();
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.register").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.register").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
       err
     };
@@ -192,8 +192,8 @@ void GitFilterRegistry::UnregisterWorker::HandleOKCallback() {
     } else {
       err = Nan::To<v8::Object>(Nan::Error("Method register has thrown an error.")).ToLocalChecked();
     }
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.unregister").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.unregister").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
       err
     };
@@ -204,8 +204,8 @@ void GitFilterRegistry::UnregisterWorker::HandleOKCallback() {
   }
   else if (baton->error_code < 0) {
     v8::Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method unregister has thrown an error.")).ToLocalChecked();
-    err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.unregister").ToLocalChecked());
+    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.unregister").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/src/promise_completion.cc b/generate/templates/manual/src/promise_completion.cc
index 1e503ac72..22203b41d 100644
--- a/generate/templates/manual/src/promise_completion.cc
+++ b/generate/templates/manual/src/promise_completion.cc
@@ -64,7 +64,7 @@ void PromiseCompletion::Setup(v8::Local<v8::Function> thenFn, v8::Local<v8::Valu
   };
 
   // call the promise's .then method with resolve and reject callbacks
-  Nan::Callback(thenFn).Call(promise, 2, argv);
+  Nan::Call(Nan::Callback(thenFn), promise, 2, argv);
 }
 
 // binds an object to be the context of the function.
diff --git a/generate/templates/manual/src/str_array_converter.cc b/generate/templates/manual/src/str_array_converter.cc
index 6100fd01d..eb75aa41d 100644
--- a/generate/templates/manual/src/str_array_converter.cc
+++ b/generate/templates/manual/src/str_array_converter.cc
@@ -9,12 +9,12 @@
 using namespace v8;
 using namespace node;
 
-git_strarray *StrArrayConverter::Convert(Local<v8::Value> val) {
+git_strarray *StrArrayConverter::Convert(v8::Local<v8::Value> val) {
   if (!Nan::To<bool>(val).FromJust()) {
     return NULL;
   }
   else if (val->IsArray()) {
-    return ConvertArray(Array::Cast(*val));
+    return ConvertArray(v8::Local<v8::Array>::Cast(val));
   }
   else if (val->IsString() || val->IsStringObject()) {
     return ConvertString(Nan::To<v8::String>(val).ToLocalChecked());
@@ -33,18 +33,18 @@ git_strarray * StrArrayConverter::AllocStrArray(const size_t count) {
   return result;
 }
 
-git_strarray *StrArrayConverter::ConvertArray(Array *val) {
+git_strarray *StrArrayConverter::ConvertArray(v8::Local<v8::Array> val) {
   git_strarray *result = AllocStrArray(val->Length());
 
   for(size_t i = 0; i < result->count; i++) {
-    Nan::Utf8String entry(val->Get(i));
+    Nan::Utf8String entry(Nan::Get(val, i).ToLocalChecked());
     result->strings[i] = strdup(*entry);
   }
 
   return result;
 }
 
-git_strarray* StrArrayConverter::ConvertString(Local<String> val) {
+git_strarray* StrArrayConverter::ConvertString(v8::Local<v8::String> val) {
   char *strings[1];
   Nan::Utf8String utf8String(val);
 
diff --git a/generate/templates/partials/async_function.cc b/generate/templates/partials/async_function.cc
index 8cd11d0f9..f5c4c8918 100644
--- a/generate/templates/partials/async_function.cc
+++ b/generate/templates/partials/async_function.cc
@@ -167,8 +167,8 @@ void {{ cppClassName }}::{{ cppFunctionName }}Worker::HandleOKCallback() {
       } else {
         err = Nan::To<v8::Object>(Nan::Error("Method {{ jsFunctionName }} has thrown an error.")).ToLocalChecked();
       }
-      err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-      err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("{{ jsClassName }}.{{ jsFunctionName }}").ToLocalChecked());
+      Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+      Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("{{ jsClassName }}.{{ jsFunctionName }}").ToLocalChecked());
       v8::Local<v8::Value> argv[1] = {
         err
       };
@@ -219,8 +219,8 @@ void {{ cppClassName }}::{{ cppFunctionName }}Worker::HandleOKCallback() {
 
         v8::Local<v8::Array> properties = Nan::GetPropertyNames(nodeObj).ToLocalChecked();
         for (unsigned int propIndex = 0; propIndex < properties->Length(); ++propIndex) {
-          v8::Local<v8::String> propName = Nan::To<v8::String>(properties->Get(propIndex)).ToLocalChecked();
-          v8::Local<v8::Value> nodeToQueue = nodeObj->Get(propName);
+          v8::Local<v8::String> propName = Nan::To<v8::String>(Nan::Get(properties, propIndex).ToLocalChecked()).ToLocalChecked();
+          v8::Local<v8::Value> nodeToQueue = Nan::Get(nodeObj, propName).ToLocalChecked();
           if (!nodeToQueue->IsUndefined()) {
             workerArguments.push(nodeToQueue);
           }
@@ -229,8 +229,8 @@ void {{ cppClassName }}::{{ cppFunctionName }}Worker::HandleOKCallback() {
 
       if (!callbackFired) {
         v8::Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method {{ jsFunctionName }} has thrown an error.")).ToLocalChecked();
-        err->Set(Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-        err->Set(Nan::New("errorFunction").ToLocalChecked(), Nan::New("{{ jsClassName }}.{{ jsFunctionName }}").ToLocalChecked());
+        Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+        Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("{{ jsClassName }}.{{ jsFunctionName }}").ToLocalChecked());
         v8::Local<v8::Value> argv[1] = {
           err
         };
diff --git a/generate/templates/partials/convert_from_v8.cc b/generate/templates/partials/convert_from_v8.cc
index a3ea193a5..7fc74a29b 100644
--- a/generate/templates/partials/convert_from_v8.cc
+++ b/generate/templates/partials/convert_from_v8.cc
@@ -47,13 +47,13 @@
   memset((void *)(((char *)from_{{ name }}) + {{ name }}.length()), 0, 1);
   {%elsif cppClassName == 'Array'%}
 
-  Array *tmp_{{ name }} = Array::Cast(*info[{{ jsArg }}]);
+  v8::Local<v8::Array> tmp_{{ name }} = v8::Local<v8::Array>::Cast(info[{{ jsArg }}]);
   from_{{ name }} = ({{ cType }})malloc(tmp_{{ name }}->Length() * sizeof({{ cType|replace '**' '*' }}));
       for (unsigned int i = 0; i < tmp_{{ name }}->Length(); i++) {
     {%--
       // FIXME: should recursively call convertFromv8.
     --%}
-      from_{{ name }}[i] = Nan::ObjectWrap::Unwrap<{{ arrayElementCppClassName }}>(Nan::To<v8::Object>(tmp_{{ name }}->Get(Nan::New(static_cast<double>(i)))).ToLocalChecked())->GetValue();
+      from_{{ name }}[i] = Nan::ObjectWrap::Unwrap<{{ arrayElementCppClassName }}>(Nan::To<v8::Object>(Nan::Get(tmp_{{ name }}, Nan::New(static_cast<double>(i))).ToLocalChecked()).ToLocalChecked())->GetValue();
       }
   {%elsif cppClassName == 'Function'%}
   {%elsif cppClassName == 'Buffer'%}
diff --git a/generate/templates/templates/nodegit.cc b/generate/templates/templates/nodegit.cc
index c1eddd328..a9d22ce6d 100644
--- a/generate/templates/templates/nodegit.cc
+++ b/generate/templates/templates/nodegit.cc
@@ -26,14 +26,11 @@
 #if (NODE_MODULE_VERSION > 48)
   v8::Local<v8::Value> GetPrivate(v8::Local<v8::Object> object,
                                       v8::Local<v8::String> key) {
-    v8::Isolate* isolate = v8::Isolate::GetCurrent();
-    v8::Local<v8::Context> context = isolate->GetCurrentContext();
-    v8::Local<v8::Private> privateKey = v8::Private::ForApi(isolate, key);
     v8::Local<v8::Value> value;
-    v8::Maybe<bool> result = object->HasPrivate(context, privateKey);
+    Nan::Maybe<bool> result = Nan::HasPrivate(object, key);
     if (!(result.IsJust() && result.FromJust()))
       return v8::Local<v8::Value>();
-    if (object->GetPrivate(context, privateKey).ToLocal(&value))
+    if (Nan::GetPrivate(object, key).ToLocal(&value))
       return value;
     return v8::Local<v8::Value>();
   }
@@ -43,10 +40,7 @@
                       v8::Local<v8::Value> value) {
     if (value.IsEmpty())
       return;
-    v8::Isolate* isolate = v8::Isolate::GetCurrent();
-    v8::Local<v8::Context> context = isolate->GetCurrentContext();
-    v8::Local<v8::Private> privateKey = v8::Private::ForApi(isolate, key);
-    object->SetPrivate(context, privateKey, value);
+    Nan::SetPrivate(object, key, value);
   }
 #else
   v8::Local<v8::Value> GetPrivate(v8::Local<v8::Object> object,
@@ -70,7 +64,7 @@ void LockMasterSetStatus(const FunctionCallbackInfo<Value>& info) {
 
   // convert the first argument to Status
   if(info.Length() >= 0 && info[0]->IsNumber()) {
-    v8::Local<v8::Int32> value = info[0]->ToInt32(v8::Isolate::GetCurrent());
+    v8::Local<v8::Int32> value = Nan::To<v8::Int32>(info[0]).ToLocalChecked();
     LockMaster::Status status = static_cast<LockMaster::Status>(value->Value());
     if(status >= LockMaster::Disabled && status <= LockMaster::Enabled) {
       LockMaster::SetStatus(status);
@@ -91,7 +85,7 @@ void LockMasterGetDiagnostics(const FunctionCallbackInfo<Value>& info) {
 
   // return a plain JS object with properties
   v8::Local<v8::Object> result = Nan::New<v8::Object>();
-  result->Set(Nan::New("storedMutexesCount").ToLocalChecked(), Nan::New(diagnostics.storedMutexesCount));
+  Nan::Set(result,Nan::New("storedMutexesCount").ToLocalChecked(), Nan::New(diagnostics.storedMutexesCount));
   info.GetReturnValue().Set(result);
 }
 
@@ -149,11 +143,11 @@ extern "C" void init(v8::Local<v8::Object> target) {
   NODE_SET_METHOD(target, "getThreadSafetyDiagnostics", LockMasterGetDiagnostics);
 
   v8::Local<v8::Object> threadSafety = Nan::New<v8::Object>();
-  threadSafety->Set(Nan::New("DISABLED").ToLocalChecked(), Nan::New((int)LockMaster::Disabled));
-  threadSafety->Set(Nan::New("ENABLED_FOR_ASYNC_ONLY").ToLocalChecked(), Nan::New((int)LockMaster::EnabledForAsyncOnly));
-  threadSafety->Set(Nan::New("ENABLED").ToLocalChecked(), Nan::New((int)LockMaster::Enabled));
+  Nan::Set(threadSafety,Nan::New("DISABLED").ToLocalChecked(), Nan::New((int)LockMaster::Disabled));
+  Nan::Set(threadSafety,Nan::New("ENABLED_FOR_ASYNC_ONLY").ToLocalChecked(), Nan::New((int)LockMaster::EnabledForAsyncOnly));
+  Nan::Set(threadSafety,Nan::New("ENABLED").ToLocalChecked(), Nan::New((int)LockMaster::Enabled));
 
-  target->Set(Nan::New("THREAD_SAFETY").ToLocalChecked(), threadSafety);
+  Nan::Set(target,Nan::New("THREAD_SAFETY").ToLocalChecked(), threadSafety);
 
   LockMaster::Initialize();
 }

From a0f0c608a2594e9c99dbd08e4dbd15b96b905e86 Mon Sep 17 00:00:00 2001
From: Peter Hunt <peter.hunt@bglgroup.co.uk>
Date: Fri, 12 Jul 2019 11:54:10 +0100
Subject: [PATCH 11/13] :art: Minor spacing for parameters in Nan::Set

---
 generate/templates/manual/clone/clone.cc         |  8 ++++----
 .../templates/manual/commit/extract_signature.cc |  4 ++--
 generate/templates/manual/filter_list/load.cc    |  8 ++++----
 generate/templates/manual/filter_source/repo.cc  |  8 ++++----
 .../manual/patches/convenient_patches.cc         |  8 ++++----
 generate/templates/manual/remote/ls.cc           |  4 ++--
 .../manual/repository/get_references.cc          |  4 ++--
 .../templates/manual/repository/get_remotes.cc   |  4 ++--
 .../manual/repository/get_submodules.cc          |  4 ++--
 .../manual/repository/refresh_references.cc      |  4 ++--
 generate/templates/manual/revwalk/commit_walk.cc |  4 ++--
 generate/templates/manual/revwalk/fast_walk.cc   |  8 ++++----
 .../manual/revwalk/file_history_walk.cc          |  8 ++++----
 generate/templates/manual/src/filter_registry.cc | 16 ++++++++--------
 generate/templates/partials/async_function.cc    |  8 ++++----
 generate/templates/templates/nodegit.cc          | 10 +++++-----
 16 files changed, 55 insertions(+), 55 deletions(-)

diff --git a/generate/templates/manual/clone/clone.cc b/generate/templates/manual/clone/clone.cc
index 6f5f79769..a7ac262dc 100644
--- a/generate/templates/manual/clone/clone.cc
+++ b/generate/templates/manual/clone/clone.cc
@@ -144,8 +144,8 @@ void GitClone::CloneWorker::HandleOKCallback() {
       } else {
         err = Nan::To<v8::Object>(Nan::Error("Method clone has thrown an error.")).ToLocalChecked();
       }
-      Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-      Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(),
+      Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+      Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(),
                Nan::New("Clone.clone").ToLocalChecked());
       v8::Local<v8::Value> argv[1] = {err};
       callback->Call(1, argv, async_resource);
@@ -195,9 +195,9 @@ void GitClone::CloneWorker::HandleOKCallback() {
       if (!callbackFired) {
         v8::Local<v8::Object> err =
             Nan::To<v8::Object>(Nan::Error("Method clone has thrown an error.")).ToLocalChecked();
-        Nan::Set(err,Nan::New("errno").ToLocalChecked(),
+        Nan::Set(err, Nan::New("errno").ToLocalChecked(),
                  Nan::New(baton->error_code));
-        Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(),
+        Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(),
                  Nan::New("Clone.clone").ToLocalChecked());
         v8::Local<v8::Value> argv[1] = {err};
         callback->Call(1, argv, async_resource);
diff --git a/generate/templates/manual/commit/extract_signature.cc b/generate/templates/manual/commit/extract_signature.cc
index 0fea1bc0a..e96d0cc7f 100644
--- a/generate/templates/manual/commit/extract_signature.cc
+++ b/generate/templates/manual/commit/extract_signature.cc
@@ -133,8 +133,8 @@ void GitCommit::ExtractSignatureWorker::HandleOKCallback()
   else if (baton->error_code < 0)
   {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Extract Signature has thrown an error.")).ToLocalChecked();
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Commit.extractSignature").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("Commit.extractSignature").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/filter_list/load.cc b/generate/templates/manual/filter_list/load.cc
index 40ce79433..fd02a44e6 100644
--- a/generate/templates/manual/filter_list/load.cc
+++ b/generate/templates/manual/filter_list/load.cc
@@ -176,8 +176,8 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
       } else {
         err = Nan::To<v8::Object>(Nan::Error("Method load has thrown an error.")).ToLocalChecked();
       }
-      Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-      Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(),
+      Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+      Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(),
                Nan::New("FilterList.load").ToLocalChecked());
       v8::Local<v8::Value> argv[1] = {err};
       callback->Call(1, argv, async_resource);
@@ -229,9 +229,9 @@ void GitFilterList::LoadWorker::HandleOKCallback() {
       if (!callbackFired) {
         v8::Local<v8::Object> err =
             Nan::To<v8::Object>(Nan::Error("Method load has thrown an error.")).ToLocalChecked();
-        Nan::Set(err,Nan::New("errno").ToLocalChecked(),
+        Nan::Set(err, Nan::New("errno").ToLocalChecked(),
                  Nan::New(baton->error_code));
-        Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(),
+        Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(),
                  Nan::New("FilterList.load").ToLocalChecked());
         v8::Local<v8::Value> argv[1] = {err};
         callback->Call(1, argv, async_resource);
diff --git a/generate/templates/manual/filter_source/repo.cc b/generate/templates/manual/filter_source/repo.cc
index fca66a754..57c2a07f7 100644
--- a/generate/templates/manual/filter_source/repo.cc
+++ b/generate/templates/manual/filter_source/repo.cc
@@ -64,8 +64,8 @@ void GitFilterSource::RepoWorker::HandleOKCallback() {
       } else {
         err = Nan::To<v8::Object>(Nan::Error("Method repo has thrown an error.")).ToLocalChecked();
       }
-      Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-      Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(),
+      Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+      Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(),
                Nan::New("FilterSource.repo").ToLocalChecked());
       v8::Local<v8::Value> argv[1] = {err};
       callback->Call(1, argv, async_resource);
@@ -75,9 +75,9 @@ void GitFilterSource::RepoWorker::HandleOKCallback() {
     } else if (baton->error_code < 0) {
       v8::Local<v8::Object> err =
           Nan::To<v8::Object>(Nan::Error("Method repo has thrown an error.")).ToLocalChecked();
-      Nan::Set(err,Nan::New("errno").ToLocalChecked(),
+      Nan::Set(err, Nan::New("errno").ToLocalChecked(),
                Nan::New(baton->error_code));
-      Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(),
+      Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(),
                Nan::New("FilterSource.repo").ToLocalChecked());
       v8::Local<v8::Value> argv[1] = {err};
       callback->Call(1, argv, async_resource);
diff --git a/generate/templates/manual/patches/convenient_patches.cc b/generate/templates/manual/patches/convenient_patches.cc
index b27b1b66e..c2403467a 100644
--- a/generate/templates/manual/patches/convenient_patches.cc
+++ b/generate/templates/manual/patches/convenient_patches.cc
@@ -101,8 +101,8 @@ void GitPatch::ConvenientFromDiffWorker::HandleOKCallback() {
     } else {
       err = Nan::To<v8::Object>(Nan::Error("Method convenientFromDiff has thrown an error.")).ToLocalChecked();
     }
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Patch.convenientFromDiff").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("Patch.convenientFromDiff").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
@@ -119,8 +119,8 @@ void GitPatch::ConvenientFromDiffWorker::HandleOKCallback() {
 
   if (baton->error_code < 0) {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("method convenientFromDiff has thrown an error.")).ToLocalChecked();
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Patch.convenientFromDiff").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("Patch.convenientFromDiff").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/remote/ls.cc b/generate/templates/manual/remote/ls.cc
index 5afcb8840..8816e0150 100644
--- a/generate/templates/manual/remote/ls.cc
+++ b/generate/templates/manual/remote/ls.cc
@@ -86,8 +86,8 @@ void GitRemote::ReferenceListWorker::HandleOKCallback()
   else if (baton->error_code < 0)
   {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Reference List has thrown an error.")).ToLocalChecked();
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Remote.referenceList").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("Remote.referenceList").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/repository/get_references.cc b/generate/templates/manual/repository/get_references.cc
index aab66d035..8f03d60e1 100644
--- a/generate/templates/manual/repository/get_references.cc
+++ b/generate/templates/manual/repository/get_references.cc
@@ -119,8 +119,8 @@ void GitRepository::GetReferencesWorker::HandleOKCallback()
   else if (baton->error_code < 0)
   {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Repository getReferences has thrown an error.")).ToLocalChecked();
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.getReferences").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.getReferences").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/repository/get_remotes.cc b/generate/templates/manual/repository/get_remotes.cc
index cfb4e0b26..9cad189a2 100644
--- a/generate/templates/manual/repository/get_remotes.cc
+++ b/generate/templates/manual/repository/get_remotes.cc
@@ -120,8 +120,8 @@ void GitRepository::GetRemotesWorker::HandleOKCallback()
   else if (baton->error_code < 0)
   {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Repository refreshRemotes has thrown an error.")).ToLocalChecked();
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.refreshRemotes").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.refreshRemotes").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/repository/get_submodules.cc b/generate/templates/manual/repository/get_submodules.cc
index e97fa6184..71de6948c 100644
--- a/generate/templates/manual/repository/get_submodules.cc
+++ b/generate/templates/manual/repository/get_submodules.cc
@@ -101,8 +101,8 @@ void GitRepository::GetSubmodulesWorker::HandleOKCallback()
   else if (baton->error_code < 0)
   {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Repository getSubmodules has thrown an error.")).ToLocalChecked();
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.getSubmodules").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.getSubmodules").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/repository/refresh_references.cc b/generate/templates/manual/repository/refresh_references.cc
index 93d85c247..8d0fe36d6 100644
--- a/generate/templates/manual/repository/refresh_references.cc
+++ b/generate/templates/manual/repository/refresh_references.cc
@@ -673,8 +673,8 @@ void GitRepository::RefreshReferencesWorker::HandleOKCallback()
   else if (baton->error_code < 0)
   {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Repository refreshReferences has thrown an error.")).ToLocalChecked();
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.refreshReferences").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("Repository.refreshReferences").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/revwalk/commit_walk.cc b/generate/templates/manual/revwalk/commit_walk.cc
index 99f011475..91fe7f38c 100644
--- a/generate/templates/manual/revwalk/commit_walk.cc
+++ b/generate/templates/manual/revwalk/commit_walk.cc
@@ -111,8 +111,8 @@ void GitRevwalk::CommitWalkWorker::HandleOKCallback() {
     free((void *)baton->error);
   } else if (baton->error_code < 0) {
     Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Revwalk commitWalk has thrown an error.")).ToLocalChecked();
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.commitWalk").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.commitWalk").ToLocalChecked());
     Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/revwalk/fast_walk.cc b/generate/templates/manual/revwalk/fast_walk.cc
index 191bc6b4f..002251852 100644
--- a/generate/templates/manual/revwalk/fast_walk.cc
+++ b/generate/templates/manual/revwalk/fast_walk.cc
@@ -94,8 +94,8 @@ void GitRevwalk::FastWalkWorker::HandleOKCallback()
       } else {
         err = Nan::To<v8::Object>(Nan::Error("Method fastWalk has thrown an error.")).ToLocalChecked();
       }
-      Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-      Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fastWalk").ToLocalChecked());
+      Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+      Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fastWalk").ToLocalChecked());
       Local<v8::Value> argv[1] = {
         err
       };
@@ -159,8 +159,8 @@ void GitRevwalk::FastWalkWorker::HandleOKCallback()
       if (!callbackFired)
       {
         Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method next has thrown an error.")).ToLocalChecked();
-        Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-        Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fastWalk").ToLocalChecked());
+        Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+        Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fastWalk").ToLocalChecked());
         Local<v8::Value> argv[1] = {
           err
         };
diff --git a/generate/templates/manual/revwalk/file_history_walk.cc b/generate/templates/manual/revwalk/file_history_walk.cc
index e55d86e9f..d8d2935df 100644
--- a/generate/templates/manual/revwalk/file_history_walk.cc
+++ b/generate/templates/manual/revwalk/file_history_walk.cc
@@ -448,8 +448,8 @@ void GitRevwalk::FileHistoryWalkWorker::HandleOKCallback()
     } else {
       err = Nan::To<v8::Object>(Nan::Error("Method fileHistoryWalk has thrown an error.")).ToLocalChecked();
     }
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fileHistoryWalk").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fileHistoryWalk").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
       err
     };
@@ -465,8 +465,8 @@ void GitRevwalk::FileHistoryWalkWorker::HandleOKCallback()
 
   if (baton->error_code < 0) {
     v8::Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method next has thrown an error.")).ToLocalChecked();
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fileHistoryWalk").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("Revwalk.fileHistoryWalk").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/manual/src/filter_registry.cc b/generate/templates/manual/src/filter_registry.cc
index 4014b4abe..67e958d03 100644
--- a/generate/templates/manual/src/filter_registry.cc
+++ b/generate/templates/manual/src/filter_registry.cc
@@ -106,8 +106,8 @@ void GitFilterRegistry::RegisterWorker::HandleOKCallback() {
     } else {
       err = Nan::To<v8::Object>(Nan::Error("Method register has thrown an error.")).ToLocalChecked();
     }
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.register").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.register").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
       err
     };
@@ -118,8 +118,8 @@ void GitFilterRegistry::RegisterWorker::HandleOKCallback() {
   }
   else if (baton->error_code < 0) {
     v8::Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method register has thrown an error.")).ToLocalChecked();
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.register").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.register").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
       err
     };
@@ -192,8 +192,8 @@ void GitFilterRegistry::UnregisterWorker::HandleOKCallback() {
     } else {
       err = Nan::To<v8::Object>(Nan::Error("Method register has thrown an error.")).ToLocalChecked();
     }
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.unregister").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.unregister").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
       err
     };
@@ -204,8 +204,8 @@ void GitFilterRegistry::UnregisterWorker::HandleOKCallback() {
   }
   else if (baton->error_code < 0) {
     v8::Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method unregister has thrown an error.")).ToLocalChecked();
-    Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-    Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.unregister").ToLocalChecked());
+    Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+    Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("FilterRegistry.unregister").ToLocalChecked());
     v8::Local<v8::Value> argv[1] = {
       err
     };
diff --git a/generate/templates/partials/async_function.cc b/generate/templates/partials/async_function.cc
index f5c4c8918..9a1ce26fe 100644
--- a/generate/templates/partials/async_function.cc
+++ b/generate/templates/partials/async_function.cc
@@ -167,8 +167,8 @@ void {{ cppClassName }}::{{ cppFunctionName }}Worker::HandleOKCallback() {
       } else {
         err = Nan::To<v8::Object>(Nan::Error("Method {{ jsFunctionName }} has thrown an error.")).ToLocalChecked();
       }
-      Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-      Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("{{ jsClassName }}.{{ jsFunctionName }}").ToLocalChecked());
+      Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+      Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("{{ jsClassName }}.{{ jsFunctionName }}").ToLocalChecked());
       v8::Local<v8::Value> argv[1] = {
         err
       };
@@ -229,8 +229,8 @@ void {{ cppClassName }}::{{ cppFunctionName }}Worker::HandleOKCallback() {
 
       if (!callbackFired) {
         v8::Local<v8::Object> err = Nan::To<v8::Object>(Nan::Error("Method {{ jsFunctionName }} has thrown an error.")).ToLocalChecked();
-        Nan::Set(err,Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
-        Nan::Set(err,Nan::New("errorFunction").ToLocalChecked(), Nan::New("{{ jsClassName }}.{{ jsFunctionName }}").ToLocalChecked());
+        Nan::Set(err, Nan::New("errno").ToLocalChecked(), Nan::New(baton->error_code));
+        Nan::Set(err, Nan::New("errorFunction").ToLocalChecked(), Nan::New("{{ jsClassName }}.{{ jsFunctionName }}").ToLocalChecked());
         v8::Local<v8::Value> argv[1] = {
           err
         };
diff --git a/generate/templates/templates/nodegit.cc b/generate/templates/templates/nodegit.cc
index a9d22ce6d..6a3fa8ef4 100644
--- a/generate/templates/templates/nodegit.cc
+++ b/generate/templates/templates/nodegit.cc
@@ -85,7 +85,7 @@ void LockMasterGetDiagnostics(const FunctionCallbackInfo<Value>& info) {
 
   // return a plain JS object with properties
   v8::Local<v8::Object> result = Nan::New<v8::Object>();
-  Nan::Set(result,Nan::New("storedMutexesCount").ToLocalChecked(), Nan::New(diagnostics.storedMutexesCount));
+  Nan::Set(result, Nan::New("storedMutexesCount").ToLocalChecked(), Nan::New(diagnostics.storedMutexesCount));
   info.GetReturnValue().Set(result);
 }
 
@@ -143,11 +143,11 @@ extern "C" void init(v8::Local<v8::Object> target) {
   NODE_SET_METHOD(target, "getThreadSafetyDiagnostics", LockMasterGetDiagnostics);
 
   v8::Local<v8::Object> threadSafety = Nan::New<v8::Object>();
-  Nan::Set(threadSafety,Nan::New("DISABLED").ToLocalChecked(), Nan::New((int)LockMaster::Disabled));
-  Nan::Set(threadSafety,Nan::New("ENABLED_FOR_ASYNC_ONLY").ToLocalChecked(), Nan::New((int)LockMaster::EnabledForAsyncOnly));
-  Nan::Set(threadSafety,Nan::New("ENABLED").ToLocalChecked(), Nan::New((int)LockMaster::Enabled));
+  Nan::Set(threadSafety, Nan::New("DISABLED").ToLocalChecked(), Nan::New((int)LockMaster::Disabled));
+  Nan::Set(threadSafety, Nan::New("ENABLED_FOR_ASYNC_ONLY").ToLocalChecked(), Nan::New((int)LockMaster::EnabledForAsyncOnly));
+  Nan::Set(threadSafety, Nan::New("ENABLED").ToLocalChecked(), Nan::New((int)LockMaster::Enabled));
 
-  Nan::Set(target,Nan::New("THREAD_SAFETY").ToLocalChecked(), threadSafety);
+  Nan::Set(target, Nan::New("THREAD_SAFETY").ToLocalChecked(), threadSafety);
 
   LockMaster::Initialize();
 }

From 04558820578180cae9ae9446569e70ac22eb97ee Mon Sep 17 00:00:00 2001
From: Peter Hunt <peter.hunt@bglgroup.co.uk>
Date: Fri, 12 Jul 2019 12:02:41 +0100
Subject: [PATCH 12/13] :recycle: Missed one Callback->Call without
 async_resource

---
 generate/templates/manual/patches/convenient_patches.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/generate/templates/manual/patches/convenient_patches.cc b/generate/templates/manual/patches/convenient_patches.cc
index c2403467a..542623f7e 100644
--- a/generate/templates/manual/patches/convenient_patches.cc
+++ b/generate/templates/manual/patches/convenient_patches.cc
@@ -129,5 +129,5 @@ void GitPatch::ConvenientFromDiffWorker::HandleOKCallback() {
     return;
   }
 
-  callback->Call(0, NULL);
+  Nan::Call(callback, 0, NULL);
 }

From e82db052c4ed9f23ba0117cea08e0eed513f1b23 Mon Sep 17 00:00:00 2001
From: Peter Hunt <peter.hunt@bglgroup.co.uk>
Date: Fri, 12 Jul 2019 12:12:21 +0100
Subject: [PATCH 13/13] :bug: Needs the reference to callback, not the pointer

---
 generate/templates/manual/patches/convenient_patches.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/generate/templates/manual/patches/convenient_patches.cc b/generate/templates/manual/patches/convenient_patches.cc
index 542623f7e..795dca506 100644
--- a/generate/templates/manual/patches/convenient_patches.cc
+++ b/generate/templates/manual/patches/convenient_patches.cc
@@ -129,5 +129,5 @@ void GitPatch::ConvenientFromDiffWorker::HandleOKCallback() {
     return;
   }
 
-  Nan::Call(callback, 0, NULL);
+  Nan::Call(*callback, 0, NULL);
 }