-
Notifications
You must be signed in to change notification settings - Fork 13
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
Peer review and testing checklist #3
Comments
@NimG98 can you do the following. I think it makes sense for you to do them. Thanks.
|
Talked to @NimG98, she will do the testing on Windows platform as she has the laptop. Thanks. |
Tested on Linux with |
Tested on Windows successfully, with |
@evelinec I changed the titles and descriptions for the guide and SEO to include "Microsoft" instead of being just "Azure", it's now "Microsoft Azure". Can you look at the changes to see if it's okay? |
Verified one successful run of the daily build test for this draft guide. Verified the SEO changes. Looks good. Completed and closed. |
Peer Review: review to be done by a peer member.
pom.xml
,server.xml
, etc files are cleancurl
command where applicableliberty-maven-plugin
orliberty-gradle-plugin
where applicablepage-tags
are used in a guide:MicroProfile, Maven, Docker, Kubernetes, Gradle, Java EE, Security
. Only these tags are visible on the website. Latest list here.attribution
statement is accurate for the guidePeer Testing: tests to be done by a peer member.
Guide’s contributor’s (if available, otherwise peer tester’s) responsibility:
diff -r start/ finish/
and there's no differencesPeer Tester’s responsibility:
The text was updated successfully, but these errors were encountered: