Skip to content

let the user know that we couldn't find any remotes#462

Merged
derrickpersson merged 1 commit into
developmentfrom
derrick/cod-1379-if-there-is-no-remote-on-a-local
Jul 22, 2022
Merged

let the user know that we couldn't find any remotes#462
derrickpersson merged 1 commit into
developmentfrom
derrick/cod-1379-if-there-is-no-remote-on-a-local

Conversation

@derrickpersson
Copy link
Copy Markdown
Contributor

Description

  • Sending an error message to the extension if any repository doesn't have any remotes to let the user know why it may not be working.

Type of Change

  • Tech Debt (refactoring, unit tests, CI changes, pre-commit hooks, etc)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Test Configuration:

  • CCL version/branch: 1.X
  • API version/branch: 1.X

Checklist:

  • I have performed a self-review of my code
  • I have used codex to provide context, particularly in hard-to-understand areas
  • Any dependent changes have been made available in downstream modules

Screenshots:

Screen Shot 2022-07-21 at 4 31 07 PM

@derrickpersson derrickpersson requested review from a team and cpul and removed request for a team July 21, 2022 23:31
@linear
Copy link
Copy Markdown

linear Bot commented Jul 21, 2022

COD-1379 If there is no remote on a local repository the user should be notified of this limitation in codex

Currently if a user opens a repository with no remotes Codex will throw an error that does not indicate why the user is not able to progress.

We should notify the user that there are no remotes on this repository and Codex requires for there to be a valid remote to operate.

@derrickpersson derrickpersson force-pushed the derrick/cod-1379-if-there-is-no-remote-on-a-local branch from 0c3a1bb to 7374aea Compare July 21, 2022 23:32
@derrickpersson derrickpersson merged commit 14ead08 into development Jul 22, 2022
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.

2 participants