-
Notifications
You must be signed in to change notification settings - Fork 58
Creating workflow to open docs issue with need-docs label #497
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
base: main
Are you sure you want to change the base?
Conversation
add automation to trigger docs issue
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA. In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR. TestInitiated test by adding the needs-docs label to this PR. You can see if an issue was open in docs-internal when this PR is merged. CLA has been signed by users: @runleonarun |
Initiated test by adding the |
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA. In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR. CLA has not been signed by users: @runleonarun |
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA. In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR. CLA has not been signed by users: @runleonarun |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me know if you need some help with the permissions. The trigger change is non-blocking.
Feel free to ping the author (@${{ github.event.pull_request.user.login }}) directly if you have any questions! | ||
Link to the merged PR: ${{ github.event.pull_request.html_url }} | ||
secrets: | ||
FISHTOWN_BOT_PAT: ${{ secrets.FA_ASSISTANT_CI_WORKFLOW_PAT_TOKEN }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this repo has access to this secret. It's not used in our public repos. Does FISHTOWN_BOT_PAT not have access to dbt-labs/docs-internal
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@emmyoop I don't see access in the Settings > Secrets & Variables > Actions for the docs public or internal repos.
I wonder if we should open the issue in the docs public repo similar to the Core workflow and just copy how we do it over there (secrets: inherit
)? Or would this repo still require access to that secret?
Co-authored-by: Emily Rockman <[email protected]>
Adding a workflow with automation that opens an issue in the dbt-labs/docs-internal repo when the
needs-docs
label is used on a pull request. The docs team will get notified when this is opened and work to document your change.Slack message with more info
Notion doc with details and problem this is solving.