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

Field names in the import_format should be in snake case #14606

Closed
maqiuyujoyce opened this issue May 15, 2023 · 2 comments
Closed

Field names in the import_format should be in snake case #14606

maqiuyujoyce opened this issue May 15, 2023 · 2 comments

Comments

@maqiuyujoyce
Copy link

maqiuyujoyce commented May 15, 2023

Background

It looks like not all field names in import_format are in snake case, but some are in camel case. E.g. https://registry.terraform.io/providers/hashicorp/google/4.65.0/docs/resources/firebase_android_app IIUC, all fields in TF resources should be in snake case and there is no exception. Could field names in import_format be fixed? I think a simple unit test should help.

What kind of contribution is this issue about?

  • MMv1-based resource

Related PR(s), if any:

N/A

Details

See background.

@rileykarson
Copy link
Collaborator

The case we ran into looked like a bug in the end- these are probably all bugs that can be filed individually, when they happen.

@github-actions
Copy link

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants