Skip to content

add --include-default-roles flag to env export terraform command#92

Draft
ocap-kirk wants to merge 1 commit intomainfrom
tk-add-optional-global-roles-flag
Draft

add --include-default-roles flag to env export terraform command#92
ocap-kirk wants to merge 1 commit intomainfrom
tk-add-optional-global-roles-flag

Conversation

@ocap-kirk
Copy link

@ocap-kirk ocap-kirk commented Mar 27, 2025

Pull Request

Description

Add support for including the default roles for the env export terraform command.
env export terraform --include-default-roles

Type of Change

  • Bug fix
  • New feature/command
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test addition/update
  • Other (please describe):

Checklist

  • I have created an issue and linked it in this PR
  • I have created a branch from main with an appropriate name (e.g., fix/issue-123, feature/new-command)
  • My code follows the project's coding style guidelines
  • I have added tests for my changes (>90% coverage of new code)
  • I have updated the documentation if necessary
  • All tests pass locally
  • Lint checks pass locally
  • I have reviewed my own code for potential issues

New Command Details (if applicable)

  • Command is placed in the src/commands directory
  • Command file contains only argument configuration and a root command component
  • Command is wrapped with the AuthProvider component
  • Command has an optional apiKey argument
  • API key scope is declared for the command
  • Documentation added to the README

Additional Notes

Screenshots/Recordings

Comment on lines +74 to +76
if (!permit) {
throw new Error('No API key provided. Please provide a key or login first.');
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

@ocap-kirk - this logic is provided by the AuthProvider, and not needed to address here

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.

[Command] Add flag to allow default roles to be created in env export terraform command

2 participants