-
Notifications
You must be signed in to change notification settings - Fork 20
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
Migrate 'Liquibase Runner' plugin docs from wiki to GitHub #3782 #30
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Fargk ! pom.xml also needs an update so that it points to a right location. Added a few extra comments/proposals
README.adoc
Outdated
* https://www.jenkins.io/security/advisory/2020-09-23/#SECURITY-1885[Stored XSS vulnerability] | ||
-- | ||
|
||
[.conf-macro .output-inline]#Provides https://www.liquibase.org/[Liquibase] build steps that evaluate liquibase changesets.# |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be nice to cleanup leftover macros
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @oleg-nenashev ,
Not sure that need to update URL because it has already pointed on new Jenkins site.
README.adoc
Outdated
[frame="none",grid="none",width="100%",cols="25%,25%,25%,25%",] | ||
|=== | ||
|image:src/docs/images/dbdoc.png[image,width=300,height=149] | ||
|image:src/docs/images/changeset-detail.png[image,width=300,height=134] | ||
|image:src/docs/images/changeset-listing.png[image,width=300,height=138] | ||
|image:src/docs/images/changeset-summary.png[image,width=300,height=194] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Renders as 2x2 to make them bigger? Too small now IMHO
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just have made as in the original confluence page.
But changed to 2x2
Hi @oleg-nenashev, |
934c523
to
90f1a24
Compare
90f1a24
to
f26d76a
Compare
@Fargk changing URL from Wiki to GitHub repo |
Changelog:
Issue #3782