Skip to content

Add repo to post request#33

Closed
chadlagore wants to merge 1 commit into
masterfrom
chad/32-get-repo-name-to-daemon
Closed

Add repo to post request#33
chadlagore wants to merge 1 commit into
masterfrom
chad/32-get-repo-name-to-daemon

Conversation

@chadlagore

Copy link
Copy Markdown
Contributor

Status: WIP

🎟️ Ticket(s): #42


👷 Changes

  • Added the repo name to the /up request
  • We punted on this because it might show up via webhook when you add the deploy key - maybe not - easy enough to add it here.
  • SUPER ALPHA. Only knows origin 😂 just use this branch for testing.

🔦 Testing Instructions

@chadlagore chadlagore requested a review from bobheadxi December 27, 2017 04:47
@chadlagore chadlagore added the pr: finalized needs review and final approval label Dec 27, 2017

@bobheadxi bobheadxi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Comment thread cmd/deploy.go
}

// TODO: Support other repo names.
origin, err := repo.Remote("origin")

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment thread cmd/deploy.go
return nil, err
}

req := UpRequest{Repo: origin.Config().URL}

@bobheadxi bobheadxi Dec 27, 2017

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be origin.Config().URLs[0] or something of the sort i think

cmd/deploy.go:173:40: origin.Config().URL undefined (type *"github.com/ubclaunchpad/inertia/vendor/gopkg.in/src-d/go-git.v4/config".RemoteConfig has no field or method URL)

fixed on my branch

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We’re having a version mismatch 🙈

@chadlagore

Copy link
Copy Markdown
Contributor Author

Handled in #30

@chadlagore chadlagore closed this Jan 2, 2018
@bobheadxi bobheadxi deleted the chad/32-get-repo-name-to-daemon branch January 7, 2018 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: finalized needs review and final approval

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants