Skip to content

Comments

fix: rename inject variable to avoid shadowing gulp-inject plugin#157

Closed
Sanskar12G wants to merge 1 commit intom-lab:mainfrom
Sanskar12G:fix/gulp-inject-variable-shadowing
Closed

fix: rename inject variable to avoid shadowing gulp-inject plugin#157
Sanskar12G wants to merge 1 commit intom-lab:mainfrom
Sanskar12G:fix/gulp-inject-variable-shadowing

Conversation

@Sanskar12G
Copy link

fixes issue : #119
The gulp-inject module was imported as 'inject' which was shadowed
by the Gulp task function also named 'inject' This caused infinite
recursion when running 'gulp inject':
RangeError: Maximum call stack size exceeded

#119 Fix
Renamed import variable from inject to gulpInject to avoid the
naming conflict

@robertodauria
Copy link
Contributor

The referenced issue has been closed, and the gulpfile is being replaced as part of #83. Please read existing issues and PRs before contributing.

Also, please review our engagement requirements and our policy on AI-assisted contributions.

Closing this PR.

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