-
Notifications
You must be signed in to change notification settings - Fork 21.5k
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
Update sql-data-warehouse-manage-monitor.md #124476
Conversation
The link was wrong
@Yazdanifard : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit ecae860: 💡 Validation status: suggestions
articles/synapse-analytics/sql-data-warehouse/sql-data-warehouse-manage-monitor.md
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. For any questions, please:
|
@WilliamDAssafMSFT Important: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
articles/synapse-analytics/sql-data-warehouse/sql-data-warehouse-manage-monitor.md
Outdated
Show resolved
Hide resolved
Hi @Yazdanifard! Thanks for taking the time to create this pull request. Nice catch. I suggested a correction to the end of the URL. Please accept the commit suggestion, and we'll get this fix merged in. |
Reminder - @Yazdanifard. Please accept the commit suggestion, and we'll get this fix merged in. |
@WilliamDAssafMSFT Thank you for your message & review. |
Almost! Unresolve the comment, then commit the suggestion. Then, we can move forward. |
…se-manage-monitor.md Co-authored-by: William Assaf MSFT <74387232+WilliamDAssafMSFT@users.noreply.github.com>
Learn Build status updates of commit aa83d85: 💡 Validation status: suggestions
articles/synapse-analytics/sql-data-warehouse/sql-data-warehouse-manage-monitor.md
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. For any questions, please:
|
Should I close this PR now? |
Learn Build status updates of commit aa83d85: 💡 Validation status: suggestions
articles/synapse-analytics/sql-data-warehouse/sql-data-warehouse-manage-monitor.md
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. For any questions, please:
|
articles/synapse-analytics/sql-data-warehouse/sql-data-warehouse-manage-monitor.md
Outdated
Show resolved
Hide resolved
@Yazdanifard sorry for the delay. Please accept the latest suggestion, then we can merge. |
…se-manage-monitor.md Co-authored-by: William Assaf MSFT <74387232+WilliamDAssafMSFT@users.noreply.github.com>
Learn Build status updates of commit 564a480: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
#sign-off |
The link was wrong