-
Notifications
You must be signed in to change notification settings - Fork 64
Style and consistency check for Medical Diagnosis #293
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
Conversation
53a9c74 to
8d4bb0c
Compare
|
@day0hero Kindly review these changes for the medical diagnosis pattern. The Getting Started needs some more work, but that should not hold you from reviewing the other chapters. In the interim, i'll submit changes for the Getting Started chapter. |
|
/lgtm |
|
New changes are detected. LGTM label has been removed. |
|
Merging this one with spellcheck errors. This will be fixed after https://issues.redhat.com/browse/TELCODOCS-1542 is resolved |
d4ab029 to
6a2f536
Compare
|
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: abhatt-rh The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
- removed opendatahub from app list since we no longer deploy it. - commented out the NOTE about active development - docs now reflect the correct secret template - docs now reflect the correct s3 bucket to sync from - changed all references of `make *` to `./pattern.sh make install` - removed references to `pattern.sh` that no longer exist (make predeploy) These are links that point to the old hybrid-cloud-patterns.io. Now they point to the new site. This should also fix our failing htmltest jobs.
|
New changes are detected. LGTM label has been removed. |
https://issues.redhat.com/browse/TELCODOCS-1023