Add Rust Summit page with event details and registration info#1022
Add Rust Summit page with event details and registration info#1022Cheukting wants to merge 2 commits intoEuroPython:ep2025from
Conversation
|
@rodrigogiraoserrao seems I do not have the right SSH key to run the preview |
No idea what this is/how to fix it. @egeakman sorry to rope you in, but do you have any idea what this issue with the preview is? |
|
@Cheukting @rodrigogiraoserrao The reason is simple. This PR is based on a fork that is not in the EPS repository. The system to create the previews does need a secret that is only accessible within the EPS repos. Had the PR been based on a branch of EuroPython/website the secret would be available and everything works as expected. |
Thanks for the explanation! Opened #1023 so we can see the preview before merging. |
|
So only folks having access to the EP repo can make PR I guess, ok
…---
Cheuk Ting Ho
🧸
Website: https://cheuk.dev
LinkedIn: https://www.linkedin.com/in/cheukting-ho
Schedule a meeting: https://calendly.com/cheuk
On Fri, 28 Feb 2025 at 01:02, Rodrigo Girão Serrão ***@***.***> wrote:
@Cheukting <https://github.com/Cheukting> @rodrigogiraoserrao
<https://github.com/rodrigogiraoserrao> The reason is simple.
This PR is based on a fork that is not in the EPS repository. The system
to create the previews does need a secret that is only accessible within
the EPS repos. Access to any secret for other repositories is most likely a
bad idea in most cases and therefore forbidden by GitHub. (Secret exposure,
execution of malicious code)
Had the PR been based on a branch of EuroPython/website the secret would
be available and everything works as expected.
Thanks for the explanation! Opened #1023
<#1023> so we can see the
preview before merging.
—
Reply to this email directly, view it on GitHub
<#1022 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AG3N26JMOC7Z3QIENPRMQ532R47XJAVCNFSM6AAAAABX7OI3TGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMOBYGU2TSNZTGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
[image: rodrigogiraoserrao]*rodrigogiraoserrao* left a comment
(EuroPython/website#1022)
<#1022 (comment)>
@Cheukting <https://github.com/Cheukting> @rodrigogiraoserrao
<https://github.com/rodrigogiraoserrao> The reason is simple.
This PR is based on a fork that is not in the EPS repository. The system
to create the previews does need a secret that is only accessible within
the EPS repos. Access to any secret for other repositories is most likely a
bad idea in most cases and therefore forbidden by GitHub. (Secret exposure,
execution of malicious code)
Had the PR been based on a branch of EuroPython/website the secret would
be available and everything works as expected.
Thanks for the explanation! Opened #1023
<#1023> so we can see the
preview before merging.
—
Reply to this email directly, view it on GitHub
<#1022 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AG3N26JMOC7Z3QIENPRMQ532R47XJAVCNFSM6AAAAABX7OI3TGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMOBYGU2TSNZTGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
@Cheukting for future references, you do have access to the EP repo |
Essentially a copy of #1022 but so that we could see the preview. --------- Co-authored-by: Cheukting <cheukting.ho@gmail.com> Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
4 participants
No description provided.