Skip to content

Conversation

@anthony-gomez-fastly
Copy link
Contributor

Change summary

Allow for future commands to add an optional -workspace-id flag that will check for FASTLY_WORKSPACE_ID in env vars if no value is supplied

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

  • Does your submission pass tests?

@anthony-gomez-fastly anthony-gomez-fastly marked this pull request as ready for review October 20, 2025 19:56
@anthony-gomez-fastly anthony-gomez-fastly requested a review from a team as a code owner October 20, 2025 19:56
Copy link
Contributor

@rcaril rcaril left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - did we need to test this locally?

@anthony-gomez-fastly
Copy link
Contributor Author

@rcaril I modified an existing function to accept the flag, and was able to print the string provided. however i also noticed some default vals that would be useful so added them as well

@anthony-gomez-fastly anthony-gomez-fastly merged commit f8b5b9a into main Oct 22, 2025
13 checks passed
@anthony-gomez-fastly anthony-gomez-fastly deleted the CDTOOL-1191-add-support-for-workspace-id branch October 22, 2025 17:04
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