Skip to content

Conversation

@juliusknorr
Copy link
Member

  • ci: FIx permissions needed for update node dist files
  • tmp: run on this branch for testing

πŸ“ Summary

  • Resolves: #

πŸ–ΌοΈ Screenshots

🏚️ Before 🏑 After
B A

🚧 TODO

  • ...

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
  • Documentation (README or documentation) has been updated or is not required

@codecov
Copy link

codecov bot commented Dec 12, 2025

Codecov Report

βœ… All modified and coverable lines are covered by tests.

πŸ“’ Thoughts on this report? Let us know!

@juliusknorr juliusknorr force-pushed the ci/fix-update-node-dist branch from c6e9246 to f0a0dd7 Compare December 12, 2025 21:17
Copy link
Collaborator

@max-nextcloud max-nextcloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for taking care of this! ❀️

Copy link
Member

@mejo- mejo- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❀️

@mejo-
Copy link
Member

mejo- commented Dec 13, 2025

/backport to stable32

@mejo-
Copy link
Member

mejo- commented Dec 13, 2025

/backport to stable31

@mejo-
Copy link
Member

mejo- commented Dec 13, 2025

Merging as test failures seem unrelated.

@mejo- mejo- merged commit cd9b903 into main Dec 13, 2025
57 of 82 checks passed
@mejo- mejo- deleted the ci/fix-update-node-dist branch December 13, 2025 09:26
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.

4 participants