Skip to content

Add CODENOTIFY file for document changes#20497

Merged
tdcmeehan merged 1 commit intoprestodb:masterfrom
tdcmeehan:codenotify
Aug 8, 2023
Merged

Add CODENOTIFY file for document changes#20497
tdcmeehan merged 1 commit intoprestodb:masterfrom
tdcmeehan:codenotify

Conversation

@tdcmeehan
Copy link
Contributor

@tdcmeehan tdcmeehan commented Aug 4, 2023

Description

Notify documentation writer @steveburnett on documentation changes for quality.

Motivation and Context

@steveburnett is a professional documentation writer and is motivated to improve our documentation by helping their review.

Impact

Notifications on document changes.

Test Plan

See tdcmeehan#3 for example notification

Contributor checklist

  • Please make sure your submission complies with our development, formatting, commit message, and attribution guidelines.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

Please follow release notes guidelines and fill in the release notes below.

If release note is NOT required, use:

== NO RELEASE NOTE ==

@tdcmeehan tdcmeehan requested a review from rschlussel August 4, 2023 18:18
@tdcmeehan tdcmeehan requested a review from a team as a code owner August 4, 2023 18:18
@tdcmeehan tdcmeehan requested a review from presto-oss August 4, 2023 18:18
@tdcmeehan tdcmeehan closed this Aug 7, 2023
@tdcmeehan tdcmeehan reopened this Aug 7, 2023
@tdcmeehan tdcmeehan merged commit 095499c into prestodb:master Aug 8, 2023
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