Skip to content

gitignore: ignore test/.npm/#22337

Merged
carlocab merged 1 commit into
Homebrew:mainfrom
HaraldNordgren:gitignore-test-npm
May 19, 2026
Merged

gitignore: ignore test/.npm/#22337
carlocab merged 1 commit into
Homebrew:mainfrom
HaraldNordgren:gitignore-test-npm

Conversation

@HaraldNordgren
Copy link
Copy Markdown
Contributor

@HaraldNordgren HaraldNordgren commented May 18, 2026

This folder came when running tests for my last PR.


  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them? Performance claims (e.g. "this is faster") must include Hyperfine benchmarks.
  • Have you written new tests (excluding integration tests) for your changes? Here's an example.
  • Have you successfully run brew lgtm (style, typechecking and tests) with your changes locally?

  • AI was used to generate or assist with generating this PR.

@request-info
Copy link
Copy Markdown

request-info Bot commented May 18, 2026

Please provide a better issue/pull request title and/or description!

@request-info request-info Bot added the needs response Needs a response from the issue/PR author label May 18, 2026
@github-actions
Copy link
Copy Markdown

Thanks for your pull request. This has been closed because it appears to use an incomplete or outdated pull request template.

Please edit the pull request body to fill in the current pull request template. This workflow will reopen the pull request automatically once the template is complete.

@github-actions github-actions Bot closed this May 18, 2026
@github-actions github-actions Bot reopened this May 18, 2026
Copy link
Copy Markdown
Member

@MikeMcQuaid MikeMcQuaid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@HaraldNordgren
Copy link
Copy Markdown
Contributor Author

@MikeMcQuaid I think you also need to press the merge button here 😃

@carlocab carlocab added this pull request to the merge queue May 19, 2026
@MikeMcQuaid
Copy link
Copy Markdown
Member

@HaraldNordgren Apologies, my children extracted me from the computer 😁

@HaraldNordgren
Copy link
Copy Markdown
Contributor Author

Apologies, my children extracted me from the computer 😁

No worries at all! Much better to spend time with the kids, they grow up fast!

Merged via the queue into Homebrew:main with commit e2b284e May 19, 2026
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs response Needs a response from the issue/PR author

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants