Skip to content

Conversation

@lolgear
Copy link
Contributor

@lolgear lolgear commented Sep 17, 2019

@lolgear
Copy link
Contributor Author

lolgear commented Sep 17, 2019

@lucasderraugh Awaiting for review :)

[self.appDelegate _openRepositoryWithURL:url withCloneMode:kCloneMode_None windowModeID:NSNotFound];
}
else {
// fill error if needed.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we report back if the item doesn't contain a git repository?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@lucasderraugh Sure.
Does this check exist in GCRepository?

@lolgear
Copy link
Contributor Author

lolgear commented Oct 13, 2019

@lucasderraugh
Strange build failure. GenerateDsym fails for iPhone target. Hmm...

@lolgear
Copy link
Contributor Author

lolgear commented Oct 14, 2019

@lucasderraugh
Ah, I figured it out. Xcode refuses to skip warnings.
Could you check and merge it?

@lolgear
Copy link
Contributor Author

lolgear commented Jan 16, 2020

Ready?

@lolgear
Copy link
Contributor Author

lolgear commented Feb 2, 2020

@lucasderraugh Ready?

@lucasderraugh lucasderraugh merged commit 7e19f2c into git-up:master Feb 4, 2020
@lucasderraugh
Copy link
Collaborator

Looks good. Sorry for taking forever to review this 😬.

simpzan pushed a commit to simpzan/GitUp that referenced this pull request Oct 22, 2020
* application: services provider has been added.

* application: app delegate services provider has been registered.

* application: services provider notes have been added.

* application: info plist service have been added.

* Fix spelling mistake

* application: services provider check for existing repository has been added.

* application: services provider imports have been fixed.

Co-authored-by: Lucas Derraugh <lucas@derraugh.com>
@lolgear lolgear deleted the services/finder_context_menu branch May 17, 2021 09:33
@lolgear lolgear restored the services/finder_context_menu branch May 18, 2021 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants