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: resolve extends layer document path #266

Merged
merged 3 commits into from
Dec 4, 2022

Conversation

cpreston321
Copy link
Contributor

Issue

issue: #264

Context

This is a fix for extending multiple layers of nuxt and not just extending the srcDir

I really wanted to get this out as it is breaking my code for creating a re-useable template using the nuxt extends future!

if you need anything please let me know!

Thanks 💚

Here is a picture of it working as intended!

image

@Diizzayy Diizzayy self-assigned this Dec 4, 2022
@codecov
Copy link

codecov bot commented Dec 4, 2022

Codecov Report

Merging #266 (f4eeaa2) into main (7e32485) will increase coverage by 0.99%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #266      +/-   ##
==========================================
+ Coverage   92.63%   93.62%   +0.99%     
==========================================
  Files           6        6              
  Lines         611      612       +1     
  Branches      119      121       +2     
==========================================
+ Hits          566      573       +7     
+ Misses         44       39       -5     
+ Partials        1        0       -1     
Impacted Files Coverage Δ
src/module.ts 89.87% <100.00%> (+1.87%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Diizzayy
Copy link
Owner

Diizzayy commented Dec 4, 2022

@cpreston321 Great job on the PR! It will be released shortly

@Diizzayy Diizzayy merged commit 035eaec into Diizzayy:main Dec 4, 2022
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.

2 participants