Skip to content
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

chore(docs): minor doc cleanup #344

Merged
merged 1 commit into from
Mar 30, 2023
Merged

Conversation

echeung-amzn
Copy link
Member

Related to #343


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@echeung-amzn echeung-amzn changed the title docs: minor doc cleanup chore(docs): minor doc cleanup Mar 30, 2023
Comment on lines -59 to -63
<details><summary><strong>Golang</strong></summary>

Coming soon!
</details>

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No longer coming soon?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It hasn't been a priority so I don't want to mislead people. There's an open issue for it though: #21

@@ -6,7 +6,9 @@ import { MetricFactory } from "../metric";
import { AwsConsoleUrlFactory } from "../url";

/**
* A scope (construct) where all monitoring constructs are living in.
* A scope where all the monitoring of constructs is managed (alarms, dashboards, etc.).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* A scope where all the monitoring of constructs is managed (alarms, dashboards, etc.).
* A scope where all monitored constructs are managed from (ie. alarms, dashboards, etc.).

@mergify mergify bot merged commit 6e04b20 into cdklabs:main Mar 30, 2023
@echeung-amzn echeung-amzn deleted the re-dashboards branch March 30, 2023 20:09
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