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

Config file path will change in the future #9

Closed
kylewillmon opened this issue Apr 25, 2022 · 0 comments · Fixed by #11
Closed

Config file path will change in the future #9

kylewillmon opened this issue Apr 25, 2022 · 0 comments · Fixed by #11
Assignees
Labels
🔥 High Priority This should be addressed immediately

Comments

@kylewillmon
Copy link
Contributor

The settings.yaml file will be moving in the next release of the CLI, which will break the method used here to set phylum_token.

Instead of modifying the settings.yaml file, this action should be setting an environment variable that can be used in future steps.

The CLI uses the PHYLUM_API_KEY environment variable (which has existed for a while, but is not yet documented until phylum-dev/cli#228 is merged), so setting this should make everything work in a way that will not break when the settings.yaml file moves.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔥 High Priority This should be addressed immediately
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants