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: Updated README with a full revamp #31614

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

Conversation

justo-mend
Copy link
Collaborator

Changes

Updated README with a full revamp.

Context

Restructuring the README file for better readability and reference to the various ways to run Renovate.

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

readme.md Show resolved Hide resolved
readme.md Show resolved Hide resolved
readme.md Show resolved Hide resolved

- Delivers update PRs directly to your repo
- Relevant package files are discovered automatically
- Pull Requests automatically generated in your repo
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- Pull Requests automatically generated in your repo

This duplicates information from the first list item: Delivers update PRs directly to your repo.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I see your point. The heading is a summary and this bullet point fleshes it out in more detail.

- Delivers update PRs directly to your repo
- Relevant package files are discovered automatically
- Pull Requests automatically generated in your repo
- Provides useful information to help you decide which updates to accept (age, adoption, pass rates, merge confidence)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- Provides useful information to help you decide which updates to accept (age, adoption, pass rates, merge confidence)
- Renovate's PRs have badges to help you decide if you want to update:
- The age of the package update
- How many of the Mend-hosted app users updated to this version of the package
- The test pass rates for Mend-hosted app users
- A merge confidence score, that combines all the info above into a single number

Copy link
Collaborator Author

@justo-mend justo-mend Sep 26, 2024

Choose a reason for hiding this comment

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

For the page re-write, I went with an approach to reduce the overall page height by deliberately collapsing bulleted lists into comma-separated strings. It's an overview page and doesn't really need to explain every detail about the badges, IMHO.

readme.md Outdated Show resolved Hide resolved

For Bitbucket Cloud, Bitbucket Server, Forgejo, Gitea and GitLab, use our self-hosting option.
We prefer all items to be opened as a discussion before opening an issue.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
We prefer all items to be opened as a discussion before opening an issue.
Please open a Discussion, to get help, suggest a new feature, or to report a bug.
We only want maintainers to open Issues.

How about this?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks for the suggestion. I'll think about this one.
In any event, can you please remove the comma after "Discussion".

readme.md Outdated Show resolved Hide resolved

## Contributing
Follow us on Twitter, LinkedIn and Mastodon.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Follow us on Twitter, LinkedIn and Mastodon.
Follow us on:

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I don't really like adding colons to a list heading.

If you want to contribute to Renovate or get a local copy running, please read the instructions in [contributing guidelines](.github/contributing.md).
To get started look at the [list of good first issues](https://github.com/renovatebot/renovate/contribute).
- Twitter: [x.com/mend_io](http://x.com/mend_io)
- LinkedIn: [linkedin.com/company/mend-io](https://linkedin.com/company/mend-io)
Copy link
Collaborator

Choose a reason for hiding this comment

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

The list is missing the Mastodon contact info. 😉

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Removed the detail from the list heading per your suggestion. Thanks

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