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

docs: add troubleshooting page #1752

Merged
merged 4 commits into from Nov 21, 2022
Merged

docs: add troubleshooting page #1752

merged 4 commits into from Nov 21, 2022

Conversation

chingor13
Copy link
Contributor

Fixes #1687

@chingor13 chingor13 requested a review from a team as a code owner November 15, 2022 22:29
@chingor13 chingor13 requested a review from a team November 15, 2022 22:29
@product-auto-label product-auto-label bot added the size: m Pull request size is medium. label Nov 15, 2022
@chingor13 chingor13 requested a review from bcoe November 16, 2022 17:54
Copy link
Contributor

@bcoe bcoe left a comment

Choose a reason for hiding this comment

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

Left some suggestions.

The easiest way to debug your `release-please` configuration is to use the bundled
CLI.

To use the CLI, you will need to use a GitHub 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.


```bash
export GITHUB_TOKEN="my-access-token"
release-please release-pr \
Copy link
Contributor

Choose a reason for hiding this comment

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

Since many folks using the tool are non JavaScript users, I would add a section on:

"Setting up your environment", which links to nvm and explains they will need to:

  1. git clone the repository.
  2. setup Node.js with nvm.
  3. npm i
  4. npm link .

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't think it's necessary to teach folks how to install node.

@chingor13 chingor13 enabled auto-merge (squash) November 21, 2022 21:30
@chingor13 chingor13 merged commit d790796 into main Nov 21, 2022
@chingor13 chingor13 deleted the troubleshooting branch November 21, 2022 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

How does release-please work out which release is the "previous" release?
2 participants