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

[stable31] chore(deps): Bump pdfjs-dist from 3.11.174 to 4.0.189 #1132

Conversation

danxuliu
Copy link
Member

@danxuliu danxuliu commented Feb 5, 2025

Backport of #1131

dependabot bot and others added 2 commits February 5, 2025 18:47
Bumps [pdfjs-dist](https://github.com/mozilla/pdfjs-dist) from 3.11.174 to 4.0.189.
- [Commits](https://github.com/mozilla/pdfjs-dist/commits)

---
updated-dependencies:
- dependency-name: pdfjs-dist
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
The "viewer.php" acts as the "viewer.html" file included in the PDF.js
package. This commit updates "viewer.php" with the changes to
"viewer.html" from PDF.js 3.11.174 to 4.0.189 while respecting the
customized parts of "viewer.php".

Signed-off-by: Daniel Calviño Sánchez <[email protected]>
@danxuliu danxuliu added 3. to review dependencies Pull requests that update a dependency file labels Feb 5, 2025
@danxuliu danxuliu added this to the Nextcloud 31 milestone Feb 5, 2025
@Altahrim Altahrim mentioned this pull request Feb 5, 2025
1 task
@AndyScherzinger
Copy link
Member

@danxuliu red CI - related

@danxuliu
Copy link
Member Author

danxuliu commented Feb 5, 2025

@danxuliu red CI - related

Yes and no :-) It is a false positive, because the check fails whenever viewer.html was modified, even if viewer.php was adjusted as needed (which it was in this pull request, see the second commit).

For context that check was added to avoid automatic merges of PDF.js version bumps without a human adjusting viewer.php (but once a human adjusts it a force merge is needed 🤷).

@AndyScherzinger AndyScherzinger merged commit 4c82028 into stable31 Feb 5, 2025
38 of 40 checks passed
@danxuliu danxuliu deleted the backport/1131/stable31-dependabot/npm_and_yarn/pdfjs-dist-4.0.189 branch February 5, 2025 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants