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

feat: add .gitignore to alr init generated test crate #1894

Merged

Conversation

AldanTanneo
Copy link
Contributor

No description provided.

@mosteo
Copy link
Member

mosteo commented Mar 14, 2025

Is this really necessary? Isn't the .gitignore of the main crate applying to the test crate too?

@mosteo
Copy link
Member

mosteo commented Mar 14, 2025

I see that the problem is that our .gitignore is not very well defined and so lots of things aren't properly ignored in subcrates.

@AldanTanneo
Copy link
Contributor Author

Yeah, it only ignores specific folders at the root of the repository, and not in subcrate. I added this as a nested .gitignore, do you think it would be better to modify the existing .gitignore?

@mosteo
Copy link
Member

mosteo commented Mar 17, 2025

No, I think it's okay to be targeted rather than risking ignoring something not created by us.

@mosteo mosteo merged commit dd463e8 into alire-project:master Mar 17, 2025
33 checks passed
@AldanTanneo AldanTanneo deleted the feature/test_crate_gitignore branch March 17, 2025 11:29
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