Skip to content

Add notes and contribution info to readme #16

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

Merged
merged 2 commits into from
Jun 20, 2019
Merged

Add notes and contribution info to readme #16

merged 2 commits into from
Jun 20, 2019

Conversation

c24t
Copy link
Member

@c24t c24t commented Jun 19, 2019

Add a stub readme with a link to the SIG meeting notes to fix #13.

Add a stub readme with a link to the SIG meeting notes to fix #13.
@c24t c24t requested a review from Oberon00 June 19, 2019 19:13
@c24t c24t mentioned this pull request Jun 19, 2019
@Oberon00
Copy link
Member

I can't find a link to the meeting itself (hangout): Neither here, nor in the community README, nor in the linked Google calendar.

Also I'm not sure we should duplicate the information from the community README. IMHO, only one of the two READMEs should document the specifics and there should be a link to the one that does in the other.

On another note, reading "At least 1 approving review is required by reviewers with write access. ", I don't think I have write access: I'm in the OpenTelemetry org, but not in any teams.

@c24t
Copy link
Member Author

c24t commented Jun 20, 2019

I can't find a link to the meeting itself (hangout)

The hangout info is in the calendar event details:

Joining info | meet.google.com/yyw-hqif-wmu
Or dial: +1 601-658-0516  PIN: 736063

but it sounds like we should switch this to zoom in any case. @carlosalberto do you have a link we can use while we're waiting for the CNCF zoom? I'll update the calendar event to include it.

only one of the two READMEs should document the specifics and there should be a link to the one that does in the other

I agree with that in principle, but I was trying to make it as easy as possible for people just browsing the repo.

I don't think I have write access

I looks like I don't have admin access to give you write access... I'll follow up in person.

@c24t c24t merged commit a1c5ab4 into master Jun 20, 2019
@c24t c24t deleted the c24t-readme-stub branch June 20, 2019 18:17
srikanthccv pushed a commit to srikanthccv/opentelemetry-python that referenced this pull request Nov 1, 2020
* Add CODEOWNERS

* Add CODEOWNERS

* rename @draffensperger
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.

Public access to meeting notes
4 participants