Skip to content

Fixing preserving linux file permissions - #531

Merged
5saviahv merged 1 commit into
cthackers:masterfrom
NickAllmakerOpsCompass:fix_fileattr
Aug 18, 2024
Merged

5saviahv merged 1 commit into
cthackers:masterfrom
NickAllmakerOpsCompass:fix_fileattr

Conversation

@NickAllmakerOpsCompass

Copy link
Copy Markdown
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
Copy Markdown

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
@chadlwilson

Copy link
Copy Markdown

Thanks!

@NickAllmakerOpsCompass
NickAllmakerOpsCompass deleted the fix_fileattr branch August 19, 2024 15:08
@NickAllmakerOpsCompass

Copy link
Copy Markdown
Contributor Author

Happy to help!

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.

3 participants