FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Added debug build instructions. by mcollina · Pull Request #349 · nodegit/nodegit · GitHub

Repository navigation

Added debug build instructions. - #349

Merged
maxkorp merged 1 commit into
masterfrom
debug-build-instructions
Jan 14, 2015
Merged

maxkorp merged 1 commit into
masterfrom
debug-build-instructions

Conversation

Copy link
Copy Markdown
Collaborator

As discussed in #331.

Copy link
Copy Markdown
Member

Thanks!

Copy link
Copy Markdown
Collaborator

Gracias amigo! :D

maxkorp commented Jan 13, 2015

Copy link
Copy Markdown
Collaborator

One concern from my corner: I'm hoping to push a pretty large restructure of the build process this afternoon, which will invalidate these particular instructions (down to a simple "npm run build-debug" or such).

maxkorp commented Jan 14, 2015

Copy link
Copy Markdown
Collaborator

Eh, that requires some bigger discussion before it can land. I'm merging this, no way did your readme change cause appveyor to fail.

maxkorp added a commit that referenced this pull request Jan 14, 2015
maxkorp merged commit ad51862 into master Jan 14, 2015
tbranyen deleted the debug-build-instructions branch January 14, 2015 01:52

Copy link
Copy Markdown
Collaborator Author

appveyor fails almost every time :(, what's is purpose once we get travis?

Copy link
Copy Markdown
Collaborator

It's the only real CI we have for windows. I can't figure out why it dies
though.

On Wed, Jan 14, 2015, 2:33 AM Matteo Collina notifications@github.com
wrote:

appveyor fails almost every time :(, what's is purpose once we get travis?

—
Reply to this email directly or view it on GitHub
#349 (comment).

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL