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

doc: add minutes for meeting 16 jan #271

Merged
merged 1 commit into from
Jan 25, 2019

Conversation

gireeshpunathil
Copy link
Member

No description provided.

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@richardlau richardlau left a comment

Choose a reason for hiding this comment

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

Not sure if there's likely to be any other types of meetings in meetings-other/ that we should distinguish the best practices meetings in the file path/filename, or if this should be under wg-meetings (e.g. in a subfolder there).


## Announcements

* No accouncements this week.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* No accouncements this week.
* No announcements this week.

* Diagnostics Best Practices Guide content definition meeting [#254](https://github.com/nodejs/diagnostics/issues/254)

* Baseline issue to discuss “Best practices” - https://github.com/nodejs/diagnostics/issues/211
* First question is how we structure the Best parctices guide.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* First question is how we structure the Best parctices guide.
* First question is how we structure the Best practices guide?

* Gireesh, one issue is that there may be a lot of deployment variations so hard to
start there. On the other hand production deployments will still need the steps to get
There.
* Michael, starting with symptom provides common basis that can be used to build base content
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Michael, starting with symptom provides common basis that can be used to build base content
* Michael, starting with symptom provides common basis that can be used to build base content.

and then having a second layer with additional guidance for specific deployment
environments.

* Next question is how to prioritize the content
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Next question is how to prioritize the content
* Next question is how to prioritize the content?

* Matheus, start with symptom then narrow down to deployment scenarios. (same as Michael)
* Gireesh, one issue is that there may be a lot of deployment variations so hard to
start there. On the other hand production deployments will still need the steps to get
There.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
There.
there.

@gireeshpunathil
Copy link
Member Author

@richardlau - thanks; I am also not sure if there could be other meetings of this class, kept it separately as I think this is different from WG meetings - the proceedings that run up to this meeting, the people that involved etc. However I am open to move the file if there are more (and strong) opinions for it.

PR-URL: nodejs#271
Reviewed-By: Matheus Marchini <mat@mmarchini.me>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Michael Dawson <Michael_Dawson@ca.ibm.com>
@gireeshpunathil gireeshpunathil merged commit 3f1c93c into nodejs:master Jan 25, 2019
@gireeshpunathil
Copy link
Member Author

landed as 3f1c93c

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