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

add api_generated/rest_specs to .gitignore #195

Closed
AbhinavGarg90 opened this issue Oct 23, 2023 · 1 comment
Closed

add api_generated/rest_specs to .gitignore #195

AbhinavGarg90 opened this issue Oct 23, 2023 · 1 comment

Comments

@AbhinavGarg90
Copy link
Contributor

What/Why

What are you proposing?

Add files containted in api_generator/rest_specs to .gitignore

What users have asked for this feature?

What problems are you trying to solve?

These files never need to be pushed back to remote. Whenever the api is generated, these files will be too. Developers have to either delete the folder or remove the index manually to conveniently be able to push back to the repository.

What is the developer experience going to be?

It should be a simple addition of a line to .gitignore

Are there any security considerations?

None

Are there any breaking changes to the API

None

What is the user experience going to be?

May be inconvenient for somebody who is trying to use it as a package.

Are there breaking changes to the User Experience?

None

Why should it be built? Any reason not to?

Improved developer experience

Any remaining open questions?

Is this a change we are open to make?

@AbhinavGarg90 AbhinavGarg90 closed this as not planned Won't fix, can't repro, duplicate, stale Oct 23, 2023
@AbhinavGarg90
Copy link
Contributor Author

I realized that we cannot remove it, since a default api needs to be available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant