Skip to content

Conversation

@nschonni
Copy link
Contributor

@nschonni nschonni commented Oct 20, 2020

Why:

Because it needs access to secrets, the only way for Fork PRs to access them, is if they come from pull_request_target

What's being changed:

Check off the following:

@nschonni nschonni requested a review from a team as a code owner October 20, 2020 21:38
@Blinc215 Blinc215 mentioned this pull request Oct 21, 2020
@github github deleted a comment Oct 22, 2020
@nschonni
Copy link
Contributor Author

Looks like #677 made this flow internal-only

@nschonni nschonni closed this Oct 22, 2020
@nschonni nschonni deleted the patch-1 branch October 22, 2020 02:22
jnidzwetzki pushed a commit to jnidzwetzki/docs that referenced this pull request Oct 6, 2022
* Add alert in promscale

* fix missing >

* Add reference pages for the promscale cli and the SQL api

* Remove alerting from this PR. It was added by mistake

Co-authored-by: Jacob Prall <prall.jacob@gmail.com>
Co-authored-by: Lana Brindley <github@lanabrindley.com>
Co-authored-by: Ryan Booz <ryan@timescale.com>
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.

2 participants