make secret key placeholder more clear

[wintonzheng]

Important

Replaced default SECRET_KEY and SKYVERN_API_KEY with "PLACEHOLDER" in config.py.

• Configuration Changes:
  • Replaced default SECRET_KEY with "PLACEHOLDER" in Settings class in config.py.
  • Replaced default SKYVERN_API_KEY with "PLACEHOLDER" in Settings class in config.py.

^{This description was created by for 55b39c0. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 55b39c0 in 9 seconds

More details

• Looked at 19 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. skyvern/config.py:39

• Draft comment:
  Consider adding a comment to remind developers to replace 'PLACEHOLDER' with a secure key in production.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR changes the default values of sensitive keys to 'PLACEHOLDER', which is a good practice for security. However, it would be better to add a comment to remind developers to replace these placeholders with actual secure values in production.

Workflow ID: wflow_V2WM4mdjjQqAUUyu

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.