Skip to content
This repository has been archived by the owner on Sep 14, 2023. It is now read-only.

fix: make npm pkg nets importable #1086

Merged
merged 2 commits into from
Jun 15, 2023
Merged

fix: make npm pkg nets importable #1086

merged 2 commits into from
Jun 15, 2023

Conversation

harrysolovay
Copy link
Contributor

@harrysolovay harrysolovay commented Jun 15, 2023

I think it makes sense for a file to take precedence over a same-named dir's mod.ts, hence, let's just exclude the nets.ts from being matched by the fs walker.

@harrysolovay harrysolovay requested a review from tjjfvi as a code owner June 15, 2023 19:44
@harrysolovay harrysolovay enabled auto-merge June 15, 2023 19:50
kratico
kratico previously approved these changes Jun 15, 2023
Copy link
Contributor

@kratico kratico left a comment

Choose a reason for hiding this comment

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

LGTM

_tasks/dnt.ts Outdated Show resolved Hide resolved
@harrysolovay harrysolovay added this pull request to the merge queue Jun 15, 2023
Merged via the queue into main with commit 9bb09af Jun 15, 2023
@harrysolovay harrysolovay deleted the fix-nets-import branch June 15, 2023 22:24
ryanleecode pushed a commit that referenced this pull request Jun 23, 2023
ryanleecode pushed a commit that referenced this pull request Jun 23, 2023
ryanleecode pushed a commit that referenced this pull request Jun 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants