Skip to content

Add skip-gitignore cli arg#1131

Open
daggy1234 wants to merge 5 commits into
wasm-bindgen:masterfrom
daggy1234:patch-1
Open

Add skip-gitignore cli arg#1131
daggy1234 wants to merge 5 commits into
wasm-bindgen:masterfrom
daggy1234:patch-1

Conversation

@daggy1234

@daggy1234 daggy1234 commented May 1, 2022

Copy link
Copy Markdown

Make sure these boxes are checked! πŸ“¦βœ…

  • You have the latest version of rustfmt installed
$ rustup component add rustfmt
  • You ran cargo fmt on the code base before submitting
  • You reference which issue is being closed in the PR text

✨✨ πŸ˜„ Thanks so much for contributing to wasm-pack! πŸ˜„ ✨✨

This fixes #728 with a simple cli flag --skip-gitignore

@kyledecot

Copy link
Copy Markdown

Just started using wasm-pack and was wanting this feature–hooray for OSS πŸŽ‰

@daggy1234

daggy1234 commented Jun 29, 2022

Copy link
Copy Markdown
Author

Thanks! Just waiting for the maintainers.

@AtHeartEngineer

Copy link
Copy Markdown

@daggy1234 @ashleygwilliams Whats the status of this?

@vankessel

Copy link
Copy Markdown

Looking for this feature as well

@stephanemagnenat

Copy link
Copy Markdown

I also need this feature!

@NellowTCS

Copy link
Copy Markdown
Contributor

I need this feature as well!

@guybedford guybedford enabled auto-merge (squash) June 8, 2026 19:25
@NellowTCS

Copy link
Copy Markdown
Contributor

@guybedford
Hi! Since this PR is from 2022 and the codebase has changed a lot, do you want me to open a new PR that adds the same --skip-gitignore flag on the current master?

I can put together a clean version if that helps!

@guybedford

Copy link
Copy Markdown
Contributor

Ahh yeah I see it's using the outdated patterns. Sure a new PR would be great thanks @NellowTCS.

@NellowTCS NellowTCS mentioned this pull request Jun 8, 2026
3 tasks
@NellowTCS

Copy link
Copy Markdown
Contributor

@guybedford I opened a new PR!
#1590

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.

Allow skipping .gitignore generation

7 participants