Skip to content

Move Jenkins Setup section back to README #53

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

whut
Copy link
Contributor

@whut whut commented Nov 5, 2020

It took me hours to finally find how to set up this plugin. And only afterwards I managed to find the Build.md file with exact instructions I needed. I believe those instructions should be directly in README so it will be easy to find them, both on GitHub and on Jenkins Plugins site.

BTW. I have two questions about things I also could update README with:

  1. I am still not sure what the "Using Gerrit Trigger Plugin" section is about. Is is about using Gerrit Trigger plugin for everything, except that instead of voting via it, you vote using the pipeline steps that Gerrit Code Review plugin provides? Or it is about using multi-branch pipeline with Gerrit Code Review plugin, adding Gerrit Trigger only as as triggering mechanism instead of Gerrit WebHook plugin.

  2. Is Gerrit git-refs-filter still needed? Especially if not using Gerrit checks plugin? Also why any of those solutions are needed? WebHook plugin probably provides all required information, so there is no need to scan all internal Gerrit branches?

lucamilanesio and others added 5 commits June 12, 2020 22:38
Change-Id: I963456df3b57bd82a589156495c8092bbb625091
Change-Id: I1cf9e902424c6d072516a093a9e1c0e284b40f62
This way the most important information, that is how to use it is
right in the README. Previously possible user woudl have to somehow
guess that user guide is inside BUILD.md:)
Issues moved above Plugin Releases, to make it easier to find.

"Integrating with the Gerrit Checks plugin" section moved to "Jenkins
Setup" from "Why a new name?".

Also fixed header level for Issues and Plugin Releases, it should be h2,
that is section title, not h1, that is page title.

Also changes some header levels
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