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

Prevent netlify cli from pulling env variables #2529

Merged

Conversation

cesarvarela
Copy link
Collaborator

@cesarvarela cesarvarela commented Jan 5, 2024

closes #2531

Copy link

netlify bot commented Jan 5, 2024

Deploy Preview for staging-aiid ready!

Name Link
🔨 Latest commit 825f57e
🔍 Latest deploy log https://app.netlify.com/sites/staging-aiid/deploys/6598a8fe85ff190007d4e244
😎 Deploy Preview https://deploy-preview-2529--staging-aiid.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.

Copy link

github-actions bot commented Jan 5, 2024

Copy link

cypress bot commented Jan 5, 2024

3 failed tests on run #1256 ↗︎

3 416 30 0 Flakiness 0

Details:

Prevent netlify cli from pulling env variables
Project: aiid-site Commit: 825f57ee0d
Status: Failed Duration: 14:27 💡
Started: Jan 6, 2024 1:25 AM Ended: Jan 6, 2024 1:40 AM
Failed  cypress/e2e/integration/pages.cy.js • 3 failed tests • Cypress e2e tests

View Output

Test Artifacts
Pages > /en/ Should not have errors Test Replay
Pages > /es/ Should not have errors Test Replay
Pages > /fr/ Should not have errors Test Replay

Review all test suite changes for PR #2529 ↗︎

Copy link

github-actions bot commented Jan 6, 2024

@kepae kepae merged commit b631943 into responsible-ai-collaborative:staging Jan 8, 2024
10 of 12 checks passed
@cesarvarela
Copy link
Collaborator Author

I've should mark this pr as draft because there is a new issue that needs to be addressed with this change, should be ready in a few.

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