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

fix ENV must have two arguments #4097

Merged
merged 1 commit into from
Oct 16, 2023
Merged

Conversation

bxy4543
Copy link
Member

@bxy4543 bxy4543 commented Oct 16, 2023

🤖 Generated by Copilot at 4ee9116

Summary

🔧📝🔒

Initialize payment service environment variables in Kubefile. This improves the documentation and usability of the sealos platform.

sealos project
improves docs and usability
empty strings for now

Walkthrough

  • Initialize environment variables for payment service with empty strings (link)
  • Add comments to explain how to set the values or use a secret management system (link)

@sealos-ci-robot
Copy link
Member

🤖 Generated by lychee action

Summary

Status Count
🔍 Total 956
✅ Successful 371
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 580
❓ Unknown 0
🚫 Errors 4

Errors per input

Errors in docs/4.0/i18n/zh-Hans/examples/baas/install-fireboom.md

Errors in docs/4.0/docs/examples/baas/install-fireboom.md

Full action output

Full Github Actions output

@lingdie lingdie merged commit 11563c7 into labring:main Oct 16, 2023
20 of 22 checks passed
@sweep-ai
Copy link

sweep-ai bot commented Oct 16, 2023

Apply Sweep Rules to your PR?

  • Apply: Leftover TODOs in the code should be handled.
  • Apply: All new business logic should have corresponding unit tests in the tests/ directory.
  • Apply: Any clearly inefficient or repeated code should be optimized or refactored.

@bxy4543 bxy4543 deleted the fix_service_build branch October 16, 2023 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants