Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fix 313 generate scripts#315

Merged
maxkorp merged 3 commits intonodegit:masternodegit/nodegit:masterfrom
graycodes:fix-313-generate-scriptsgraycodes/nodegit:fix-313-generate-scriptsCopy head branch name to clipboard
Dec 5, 2014
Merged

Fix 313 generate scripts#315
maxkorp merged 3 commits intonodegit:masternodegit/nodegit:masterfrom
graycodes:fix-313-generate-scriptsgraycodes/nodegit:fix-313-generate-scriptsCopy head branch name to clipboard

Conversation

@graycodes
Copy link
Contributor

Updated the contributing file to reflect script changes.
Updated each generate script to pass an IIFE to module.exports.
Updated generate/index.js to account for the above.

@johnhaley81
Copy link
Collaborator

Going to wait for 310 to finish before we merge this in.

@maxkorp
Copy link
Collaborator

maxkorp commented Dec 5, 2014

Hey, now that 310 is merged in, if you can rebase this up on top of master we can go ahead and merge it in.

@graycodes graycodes force-pushed the fix-313-generate-scripts branch from 3b4d950 to 3006ad2 Compare December 5, 2014 07:31
@graycodes
Copy link
Contributor Author

Rebased against master. Waiting for build.

@maxkorp
Copy link
Collaborator

maxkorp commented Dec 5, 2014

I'm calling this passing and merging it in. See #316

maxkorp added a commit that referenced this pull request Dec 5, 2014
@maxkorp maxkorp merged commit e3216b4 into nodegit:master Dec 5, 2014
@maxkorp
Copy link
Collaborator

maxkorp commented Dec 5, 2014

Thanks for the help @xinumbralis!!

@graycodes
Copy link
Contributor Author

No worries at all! I'll get started on the tests before long.

@graycodes graycodes deleted the fix-313-generate-scripts branch December 5, 2014 14:49
@johnhaley81
Copy link
Collaborator

This fixes #313

@johnhaley81
Copy link
Collaborator

Very much appreciated @xinumbralis!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.