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

Log when the process is not authenticated and the plugin isn't running #82

Open
JessicaSachs opened this issue Sep 20, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@JessicaSachs
Copy link

Expected Behavior

Any indication that the code from the plugin is unable to be executed.

Actual Behavior

Silence 😢

Steps to Reproduce the Problem

  1. Don't set the requisite environment variables (because you didn't know any better ❤️)
  2. Run your tests and pray
  3. See that there's a check for two undocumented (at least within this plugin) process.env variables
Screenshot 2024-09-20 at 6 23 40 PM

Note

Other parts of the sauce ecosystem seem to allow you to opt in and out of using config files vs. environment variables to run this stuff. My configuration files are fine, but there's nowhere to specify them in the Sauce Plugin options.

You could consider adding the ability to pass in a sauce config file, but at the very least, please please throw a console.error in there to tell people to set the variables. 😄

@alexplischke alexplischke added the enhancement New feature or request label Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants