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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: describe intended ways to use Renovate #11197

Merged

Conversation

HonkingGoose
Copy link
Collaborator

@HonkingGoose HonkingGoose commented Aug 10, 2021

Changes:

  • Describe intended way to use Renovate in our design-decisions.md docs
  • Add section to the FAQ for end-users that describes the intended ways to use Renovate bot (Decided to drop this idea!)

Context:

Me and @viceice discussed the Renovate API not being stable (#11147).
I discovered we basically don't mention our API being unstable anywhere, we also don't really say to the end-user "Here's the supported ways to use Renovate".

@viceice made the point that it's undocumented for a reason: so that people don't go use the internals of Renovate for their own thing. Also documenting the API is a lot of work, and those API docs would need to be updated regularly.
Basically their point boils down to: "We don't document it anywhere, so people can't use it in bad ways."

I think we should at least mention somewhere suitable that our API is meant for our use only, and that people should only use Renovate as an app. We should also say which ways of using Renovate as an app are supported.

Maybe I'm honking up the wrong tree here, and the Renovate maintainers are happy to leave things as they are. 馃槈
I'll take the risk of a rejected PR, as my point will be clearer to see with a PR than with a new issue that re-hashes the discussion. 馃槃

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 tick 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

@HonkingGoose
Copy link
Collaborator Author

I have a feeling this PR will be superseded by #11202 that @rarkins is working on... 馃槃

I'll let @rarkins decide if they want this PR or not. Or maybe they can incorporate the relevant changes in their own PR. 馃槈

@rarkins
Copy link
Collaborator

rarkins commented Aug 10, 2021

I think this is correctly placed in the "development" side of the docs and complementary

docs/usage/faq.md Outdated Show resolved Hide resolved
@HonkingGoose HonkingGoose marked this pull request as ready for review September 27, 2021 07:30
docs/development/design-decisions.md Outdated Show resolved Hide resolved
docs/development/design-decisions.md Outdated Show resolved Hide resolved
Co-authored-by: Rhys Arkins <rhys@arkins.net>
rarkins and others added 2 commits September 27, 2021 13:06
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
@rarkins rarkins enabled auto-merge (squash) September 27, 2021 11:07
@rarkins rarkins merged commit fc36b27 into renovatebot:main Sep 27, 2021
@HonkingGoose HonkingGoose deleted the docs/document-intended-usage-of-renovate branch September 27, 2021 11:16
@renovate-release
Copy link
Collaborator

馃帀 This PR is included in version 27.19.3 馃帀

The release is available on:

Your semantic-release bot 馃摝馃殌

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants