fix: totalEntries is not defined by sidartaveloso · Pull Request #535 · cthackers/adm-zip · GitHub
Skip to content

fix: totalEntries is not defined - #535

Merged
cthackers merged 1 commit into
cthackers:masterfrom
devix-tecnologia:patch-1
Aug 30, 2024
Merged

cthackers merged 1 commit into
cthackers:masterfrom
devix-tecnologia:patch-1

Conversation

@sidartaveloso

Copy link
Copy Markdown
Contributor

Update zipFile.js, declaring totalEntries variable using 'let' keyword.

fix #534

Update zipFile.js, declaring totalEntries variable using 'let' keyword.

resolv: cthackers#534
@dario617

Copy link
Copy Markdown

@sidartaveloso

sidartaveloso commented Aug 29, 2024 •

Copy link
Copy Markdown
Contributor Author

@cthackers
cthackers merged commit 887f440 into cthackers:master Aug 30, 2024
chgeo referenced this pull request in capire/docs Sep 2, 2024
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [adm-zip](https://togithub.com/cthackers/adm-zip) | [`0.5.15` ->
`0.5.16`](https://renovatebot.com/diffs/npm/adm-zip/0.5.15/0.5.16) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/adm-zip/0.5.16?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/adm-zip/0.5.16?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/adm-zip/0.5.15/0.5.16?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/adm-zip/0.5.15/0.5.16?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>cthackers/adm-zip (adm-zip)</summary>

###
[`v0.5.16`](https://togithub.com/cthackers/adm-zip/releases/tag/v0.5.16)

[Compare
Source](https://togithub.com/cthackers/adm-zip/compare/v0.5.15...v0.5.16)

#### What's Changed

- Fixing preserving linux file permissions by
[@&#8203;NickAllmakerOpsCompass](https://togithub.com/NickAllmakerOpsCompass)
in
[https://github.com/cthackers/adm-zip/pull/531](https://togithub.com/cthackers/adm-zip/pull/531)
- fix: totalEntries is not defined by
[@&#8203;sidartaveloso](https://togithub.com/sidartaveloso) in
[https://github.com/cthackers/adm-zip/pull/535](https://togithub.com/cthackers/adm-zip/pull/535)

#### New Contributors

-
[@&#8203;NickAllmakerOpsCompass](https://togithub.com/NickAllmakerOpsCompass)
made their first contribution in
[https://github.com/cthackers/adm-zip/pull/531](https://togithub.com/cthackers/adm-zip/pull/531)
- [@&#8203;sidartaveloso](https://togithub.com/sidartaveloso) made their
first contribution in
[https://github.com/cthackers/adm-zip/pull/535](https://togithub.com/cthackers/adm-zip/pull/535)

**Full Changelog**:
cthackers/adm-zip@v0.5.15...v0.5.16

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/cap-js/docs).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC41Ni4wIiwidXBkYXRlZEluVmVyIjoiMzguNTYuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

ReferenceError: totalEntries is not defined.

3 participants