Skip to content

Update output.txt#2

Open
zzi9397 wants to merge 1 commit intoslavpetrov:masterfrom
zzi9397:patch-1
Open

Update output.txt#2
zzi9397 wants to merge 1 commit intoslavpetrov:masterfrom
zzi9397:patch-1

Conversation

@zzi9397
Copy link

@zzi9397 zzi9397 commented Sep 8, 2015

my first check-in

@slavpetrov
Copy link
Owner

You will need to fork the repository. You can't submit to the master
repository since then all the commits from all students would collide.

On Tue, Sep 8, 2015 at 6:16 PM zzi9397 notifications@github.com wrote:

my first check-in

You can view, comment on, or merge this pull request online at:

#2
Commit Summary

  • Update output.txt

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#2.

nasser-zalmout added a commit to nasser-zalmout/stat-nlp-nyu that referenced this pull request Sep 9, 2015
@Devking
Copy link

Devking commented Sep 19, 2015

@zzi9397 just a note that a "pull request" (often just referred to as a PR) isn't used to do a pull to your local repo. (You've probably figured that out since finishing hw1.)

However, since you're here, I thought it would be important to note that PRs are actually very important in industry, if you have code you want to push up to some master or remote repo, but need to go through a review process for it first.

You can read more about PRs here, and after doing some web dev work with a company that used GH extensively, I realized that knowing the workflow regarding PRs beforehand can really help. Hope that helps. 😄

aaronmckinstry706 added a commit to aaronmckinstry706/stat-nlp-nyu that referenced this pull request Sep 20, 2015
arpitjain2811 added a commit to arpitjain2811/stat-nlp-nyu that referenced this pull request Sep 24, 2015
nasser-zalmout added a commit to nasser-zalmout/stat-nlp-nyu that referenced this pull request Sep 29, 2015
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