Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

more fixing tips #399

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

more fixing tips #399

wants to merge 7 commits into from

Conversation

atbida
Copy link
Collaborator

@atbida atbida commented Sep 27, 2024

No description provided.

Copy link
Contributor

@alexanderahn alexanderahn left a comment

Choose a reason for hiding this comment

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

Lmk if you have any questions about my comments.

---

Sometimes there is an error when Prophecy tries to reach Git. This could be caused by your user's [authorization](#debugging-git-credentials), a [repository setting](#debugging-repository-setttings), or a [network](#debugging-network-issues) issue. Fear not! This guide outlines what to do if you see error messages such as:
`not authorized`
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: to make it easier to read in this file, add empty spaces, so:

This guide outlines what to do if you see error messages such as:

not authorized
git-upload-pack-not-permitted

or

git-recieve-pack-not-permitted

Copy link
Contributor

Choose a reason for hiding this comment

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

Or better yet, use an unordered list for these, so:

  • not authorized
  • git-upload-pack-not-permitted
  • git-recieve-pack-not-permitted

@@ -0,0 +1,57 @@
---
title: Git Troubleshooting
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: are you okay with using sentence casing for titles and section headers? We have a mix of either that or title case in our docset, and I want to start enforcing one for consistency.

Git troubleshooting

Debugging Git credentials

Debugging network issues


Sometimes your personal access token (PAT) has expired or lacks a certain permission.

1. Check if your PAT has expired. If so, create a new one with sufficient read and write access to the repository of interest. Here are the instructions to locate existing tokens and create new tokens for [Github](https://docs.github.com/en/enterprise-cloud@latest/authentication/keeping-your-account-and-data-secure/managing-your-personal-access-tokens), [Bitbucket](https://support.atlassian.com/bitbucket-cloud/docs/repository-access-tokens/), or [GitLab](https://docs.gitlab.com/ee/user/profile/personal_access_tokens.html#create-a-personal-access-token).
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be mention where the reader can check to see if their PAT has expired?

Copy link
Contributor

Choose a reason for hiding this comment

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

Is it the Prophecy Access Tokens tab?

tags: []
---

Sometimes there is an error when Prophecy tries to reach Git. This could be caused by your user's [authorization](#debugging-git-credentials), a [repository setting](#debugging-repository-setttings), or a [network](#debugging-network-issues) issue. Fear not! This guide outlines what to do if you see error messages such as:
Copy link
Contributor

Choose a reason for hiding this comment

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

We should use second person, so "This could be cause by your [authorization]..."


1. Check if your PAT has expired. If so, create a new one with sufficient read and write access to the repository of interest. Here are the instructions to locate existing tokens and create new tokens for [Github](https://docs.github.com/en/enterprise-cloud@latest/authentication/keeping-your-account-and-data-secure/managing-your-personal-access-tokens), [Bitbucket](https://support.atlassian.com/bitbucket-cloud/docs/repository-access-tokens/), or [GitLab](https://docs.gitlab.com/ee/user/profile/personal_access_tokens.html#create-a-personal-access-token).
2. Open a terminal from a computer that can access your Git repository. If you need help with this step, please reach out to your Git administrator.
3. Have the new PAT ready; you'll use it in place of a password.
Copy link
Contributor

Choose a reason for hiding this comment

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

If the reader doesn't have a new PAT, should we tell them where/how they can create one? Or link them to https://docs.prophecy.io/metadata/prophecyAPI/#generate-a-pat

1. Check if your PAT has expired. If so, create a new one with sufficient read and write access to the repository of interest. Here are the instructions to locate existing tokens and create new tokens for [Github](https://docs.github.com/en/enterprise-cloud@latest/authentication/keeping-your-account-and-data-secure/managing-your-personal-access-tokens), [Bitbucket](https://support.atlassian.com/bitbucket-cloud/docs/repository-access-tokens/), or [GitLab](https://docs.gitlab.com/ee/user/profile/personal_access_tokens.html#create-a-personal-access-token).
2. Open a terminal from a computer that can access your Git repository. If you need help with this step, please reach out to your Git administrator.
3. Have the new PAT ready; you'll use it in place of a password.
4. Check for read access by testing your ability to [clone](https://git-scm.com/docs/git-clone) a repo
Copy link
Contributor

Choose a reason for hiding this comment

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

For 4. and 5. I would add :'s at the end of the statements. And maybe use code blocks for better formatting and easier copy/paste-ability


## Debugging Repository Settings

Sometimes a repository settings can restrict a user's permissions outside the user's profile settings page.
Copy link
Contributor

Choose a reason for hiding this comment

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

"...restrict your permissions"


## Debugging Repository Settings

Sometimes a repository settings can restrict a user's permissions outside the user's profile settings page.
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure where the reader would go to check the following restrictions. We could add a line saying something like, "On the repository's security settings, check for the following restrictions."

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