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

fix: ignore node_modules declarative assets #3709

Merged
merged 1 commit into from
May 26, 2021

Conversation

joshgummersall
Copy link
Contributor

@joshgummersall joshgummersall commented May 26, 2021

Fixes #3710

@coveralls
Copy link

coveralls commented May 26, 2021

Pull Request Test Coverage Report for Build 879125975

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 85.338%

Totals Coverage Status
Change from base Build 872800976: 0.0%
Covered Lines: 19233
Relevant Lines: 21433

💛 - Coveralls

@joshgummersall joshgummersall merged commit d57152a into main May 26, 2021
@joshgummersall joshgummersall deleted the jpg/runtime-ignore-node-modules branch May 26, 2021 22:20
joshgummersall added a commit that referenced this pull request May 27, 2021
* fix: named pipe issues (#3627)

* fix: named pipe issues

- Remove unnecessary autoReconnect logic
- Simplify connection management
- Properly support client reconnection to incoming/outgoing servers

* fix: PR comments, failing tests

* fix: "lock" sockets to mitigate split brain

* test: no split brain

* fix: reject listening only if server is listening

* fix: naming for clarity

* fix: clarify onListen doc string

* feat: retry helper, use for named pipes

* fix: handle retry edge cases

* bump: adal-node to 0.2.2 (#3705)

Fixes #3704

* fix: ignore node_modules declarative assets (#3709)

See microsoft/BotFramework-Composer#7916 for details.
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.

Ignore node_modules declarative assets in runtime
3 participants