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.

Sign up for GitHub

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

Jump to bottom

[ENG-118] Imports are not supported in infisical export command #1015

Closed
OmMewt opened this issue Sep 22, 2023 · 5 comments · Fixed by #1450
Closed

[ENG-118] Imports are not supported in infisical export command #1015

OmMewt opened this issue Sep 22, 2023 · 5 comments · Fixed by #1450
Labels
🪓 cli good first issue Good for newcomers

Comments

@OmMewt
Copy link

OmMewt commented Sep 22, 2023 •
edited by vmatsiiako
Loading

When using infisical export --path={my_path} > .env it only exports the base directory envs, not the imports.

From SyncLinear.com | ENG-118
@vmatsiiako vmatsiiako changed the title Imports are not supported in infisical export command [ENG-118] Imports are not supported in infisical export command Sep 22, 2023
@akhilmhdh akhilmhdh added good first issue Good for newcomers 🪓 cli labels Sep 22, 2023
@maidul98 maidul98 added 🐞 bug Something isn't working and removed 🐞 bug Something isn't working labels Sep 22, 2023
@pratik-codes
Copy link

@akhilmhdh @maidul98 I can work on this if open to be picked up.

@akhilmhdh
Copy link
Member

Ohh sure @pratik-codes i think this would be easier too. Up for take indeed.

@maidul98

@maidul98
Copy link
Collaborator

Hey @pratik-codes let's close out #941 before picking up another issue

@akhilmhdh akhilmhdh mentioned this issue Sep 26, 2023
Closed
This was referenced Feb 22, 2024
Open
Merged
@mich040m
Copy link

I can see the PR #1450 , has been in review for 2 months now? Any idea when it will be looked into? :)

@akhilmhdh akhilmhdh linked a pull request May 15, 2024 that will close this issue
feat(cli): support of include-imports in export command #1450
Merged
5 tasks
@Cristobal-M
Copy link

Hi @mich040m according to this PR #1663 include-imports is already supported. Was merged one month ago.

I removed original commits from my branch and left one with documentation changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🪓 cli good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(cli): support of include-imports in export command Cristobal-M/infisical
6 participants
@maidul98 @Cristobal-M @akhilmhdh @mich040m @pratik-codes @OmMewt