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

RFC: debuggability metrics #49

Closed
wants to merge 6 commits into from
Closed

RFC: debuggability metrics #49

wants to merge 6 commits into from

Conversation

Kadino
Copy link
Collaborator

@Kadino Kadino commented Aug 19, 2022

Migrating #47 to a pull request for review and refinement.

Signed-off-by: Kadino sweeneys@amazon.com

Migrating #43 to a pull request for review and refinement.

Signed-off-by: Kadino <sweeneys@amazon.com>
@Kadino Kadino requested a review from a team August 19, 2022 21:54
enable word wrap in GHI diff for reviewers

Signed-off-by: Kadino <sweeneys@amazon.com>
still no word wrap?

Signed-off-by: Kadino <sweeneys@amazon.com>
Copy link
Collaborator Author

@Kadino Kadino left a comment

Choose a reason for hiding this comment

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

(delete incorrect comment)

Signed-off-by: Kadino <sweeneys@amazon.com>
rfcs/debuggability_metrics Outdated Show resolved Hide resolved
Signed-off-by: Kadino <sweeneys@amazon.com>
@jckand
Copy link
Contributor

jckand commented Aug 23, 2022

General NIT: RFC and file itself are named "debuggability" while all references in the actual RFC appear to use "debugability". Not sure which one is correct, but should be updated for consistency.

@Kadino
Copy link
Collaborator Author

Kadino commented Sep 14, 2022

Reminder for @Kadino and @AMZN-scspaldi to revise this RFC.

@Kadino Kadino marked this pull request as draft September 14, 2022 23:20
@Kadino
Copy link
Collaborator Author

Kadino commented Nov 15, 2022

Closing stale PR

@Kadino Kadino closed this Nov 15, 2022
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.

4 participants