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 secureCodeBox project to landscape #3565

Merged
merged 1 commit into from
Oct 23, 2023

Conversation

Weltraumschaf
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Oct 17, 2023

Deploy Preview for landscape ready!

Name Link
🔨 Latest commit 575c52c
🔍 Latest deploy log https://app.netlify.com/sites/landscape/deploys/65362dbd7475770007f7297a
😎 Deploy Preview https://deploy-preview-3565--landscape.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Weltraumschaf
Copy link
Contributor Author

Hi there 👋

I guess, the Netlify errors in the pipeline are because my fork has no rights to do things on your Netlify, right?

landscape.yml Outdated Show resolved Hide resolved
@castrojo
Copy link
Member

Yeah looks like they timed out, let's get past the first yml check and then I'll dig into the netlify logs. Thanks!

@Weltraumschaf
Copy link
Contributor Author

@castrojo is this caused by my changes? 🤔

[406] http://www.amplifypartners.com/ | Failed: Network error: Not Acceptable

@castrojo
Copy link
Member

No that one's not us! This is the current error:

3:44:44 AM: FATAL: secureCodeBox either has no crunchbase entry or it is invalid

Do you have one of these?

@Weltraumschaf
Copy link
Contributor Author

We don't have Crunchbase (didn't knew it until now). I simply omitted the key crunchbase in the YAML. Is this wrong?

Signed-off-by: Sven Strittmatter <sven.strittmatter@iteratec.com>
@Weltraumschaf
Copy link
Contributor Author

Ok, I learned that there is a crunchbase URL to our organisation. No idea where this came from and for what it is good for, but I added it.

@castrojo castrojo added this pull request to the merge queue Oct 23, 2023
Merged via the queue into cncf:master with commit ffd58ac Oct 23, 2023
7 checks passed
@Weltraumschaf Weltraumschaf deleted the add_securecodebox branch October 23, 2023 14:00
max8899 pushed a commit to apecloud-inc/landscape that referenced this pull request Mar 18, 2024
Signed-off-by: Sven Strittmatter <sven.strittmatter@iteratec.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