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

Fixing preserving linux file permissions #531

Merged
merged 1 commit into from
Aug 18, 2024

Conversation

NickAllmakerOpsCompass
Copy link
Contributor

Resolving the issue where fileAttr incorrectly parses as 0 when it should be a value for certain values of attr. This should resolve the issue with preserving file permissions in 0.5.15 (#530)

@chadlwilson
Copy link

Hiya @cthackers or @5saviahv - could you please take a look at this PR?

#518 and 0.5.15 seems to have regressed this behaviour from 0.5.14.

@5saviahv 5saviahv merged commit ca54d1a into cthackers:master Aug 18, 2024
11 checks passed
@chadlwilson
Copy link

Thanks!

@NickAllmakerOpsCompass NickAllmakerOpsCompass deleted the fix_fileattr branch August 19, 2024 15:08
@NickAllmakerOpsCompass
Copy link
Contributor Author

Happy to help!

chgeo pushed a commit to cap-js/docs that referenced this pull request 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
[cthackers/adm-zip#531
- fix: totalEntries is not defined by
[@&#8203;sidartaveloso](https://togithub.com/sidartaveloso) in
[cthackers/adm-zip#535

#### New Contributors

-
[@&#8203;NickAllmakerOpsCompass](https://togithub.com/NickAllmakerOpsCompass)
made their first contribution in
[cthackers/adm-zip#531
- [@&#8203;sidartaveloso](https://togithub.com/sidartaveloso) made their
first contribution in
[cthackers/adm-zip#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.

3 participants