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 Gleam (1.5.x) gitignore #4523

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vidbina
Copy link

@vidbina vidbina commented Oct 17, 2024

this is useful for developers that init a gleam project into an existing git repo where the gleam new tool will refrain from populating the .gitignore file. This situation may occur for users of devenv.nix or flake.nix.

Reasons for making this change:
There is no clear way to force the gleam new CLI tool to populate a valid .gitignore when running the CLI tool from inside a git repo. Since I manage my build tooling with devenv.nix which already spawns a git repo, I needed a different way to figure out how to get a sufficiently decent default .gitignore for my Gleam project. As I didn't find anything or any relevant open PR (searched for any PR matching the filter: gleam) to inform this, I reckoned that having this recorded in the .gitignore repo provides folks a reference in case they run into a similar problem.

Links to documentation supporting these rule changes:

this is useful for developers that init a gleam project into an existing
git repo where the `gleam new` tool will refrain from populating the
.gitignore file. This situation may occur for users of devenv.nix or
flake.nix.
@vidbina vidbina changed the title feat: Add Gleam 1.5.x gitignore feat: Add Gleam (1.5.x) gitignore Oct 17, 2024
@vidbina
Copy link
Author

vidbina commented Nov 6, 2024

Hiya @dooleydevin lmk what I can do here to make the path to merging this easier for you. Afaict, this gitignore should cover things as I haven't encountered any need to amend it yet.

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.

1 participant