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

Add a post-merge workflow to ask contributors for filling out a survey #5719

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

svrnm
Copy link
Member

@svrnm svrnm commented Dec 3, 2024

This contributes to open-telemetry/sig-contributor-experience#28

@open-telemetry/sig-contributor-experience-approvers want to try out if we can encourage new contributors to fill out a survey post merge to share with us some of their experience. The issue above provides more details

@svrnm svrnm marked this pull request as ready for review December 16, 2024 18:16
@svrnm svrnm requested a review from a team as a code owner December 16, 2024 18:16
@svrnm svrnm requested review from a team and musingvirtual and removed request for a team and musingvirtual December 16, 2024 18:17
if: env.MEMBER_FOUND == 'false'
run: |
PR_NUMBER=${{ github.event.pull_request.number }}
gh pr comment $PR_NUMBER --body "Thank you for your contribution! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by filling out this survey: https://forms.gle/WV58koUBGSG9HBY66"
Copy link
Contributor

Choose a reason for hiding this comment

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

just adding a comment here to make the link is updated to the final form, or that the form is updated (currently giving a This form can only be viewed by users in the owner's organization. )

Copy link
Contributor

Choose a reason for hiding this comment

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

The form now matches the structure we discussed during the SIG today. I updated the title to be "OpenTelemetry.io Post-Merge Survey" to keep things limited to this repo.

Copy link
Member Author

Choose a reason for hiding this comment

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

there was a restriction turned on that only members of the otel org at google can open it, which is only the admin account. I changed that and that link should work now :-)

remove duplicate section
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants