René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"(Bug) Revert.revert requires a RevertOptions parameter","articleBody":"Revert.revert requires a Revert Options parameter or a null to be passed into the method unlike other methods such as Merge.merge https://github.com/nodegit/nodegit/blob/6ff67a883c66a1cee23feb53b5f82c36c8a4df04/lib/merge.js#L28-L39\r\nand Revert.commit https://github.com/nodegit/nodegit/blob/6ff67a883c66a1cee23feb53b5f82c36c8a4df04/lib/revert.js#L16-L29. \r\n\r\nWhen a RevertOptions parameter is not passed into Revert.revert this error is thrown\r\n`(node:46081) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Callback is required and must be a Function.` when talking with @rcjsuen in slack he noticed that the error was misleading and originated from the c++ layer.\r\n\r\nHere is the code that threw this error\r\n```\r\n/**\r\n * @description reverts the git branch to the requested commit\r\n * @param {String} hash represents the hash of the commit\r\n * @async\r\n */\r\nasync function revert(hash) {\r\n\treturn new Promise(async function handlePromise(resolve, reject) {\r\n\t\ttry {\r\n\t\t\tlet repo = await openRepository(sitePath);\r\n\r\n\t\t\tlogger.info('Setting new head...');\r\n\t\t\tlet newHead = await repo.getCommit(hash);\r\n\r\n\t\t\tlogger.info('Reverting...');\r\n\t\t\tawait Git.Revert.revert(\r\n\t\t\t\trepo,\r\n\t\t\t\tnewHead,\r\n\t\t\t);\r\n\t\t\treturn resolve();\r\n\t\t} catch(err) {\r\n\t\t\tlogger.error(`${err}`);\r\n\t\t\treturn reject(err);\r\n\t\t}\r\n\t});\r\n}\r\n```","author":{"url":"https://github.com/ecoulson","@type":"Person","name":"ecoulson"},"datePublished":"2017-11-26T10:18:08.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":3},"url":"https://github.com/1403/nodegit/issues/1403"}