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

feat(pdfjs): upgrade to v4 #1115

Closed
wants to merge 2 commits into from
Closed

Conversation

DerekLiang
Copy link

@DerekLiang DerekLiang commented Jul 25, 2024

I am not sure how to update someone else's fork. So I just fork and make a new PR and hopefully it won't cause any confusing.

see discussion in #1105 . @joh-klein please review

Thank you!

Jordan Hall and others added 2 commits June 13, 2024 13:25
Signed-off-by: Jordan Hall <Jordan.Hall@mhr.co.uk>
@shamoon
Copy link
Contributor

shamoon commented Jul 27, 2024

What is the purpose of this? It’s a duplicate of #1105

@DerekLiang
Copy link
Author

I saw the #1105 was not moving forward and I was not familiar with how to contribute based on someone else's fork. Maybe someone can move forward #1105 with the recommended fix. So I can close this PR. Thank you!

@shamoon
Copy link
Contributor

shamoon commented Jul 30, 2024

You can read about how to contribute on GitHub or git in general. The maintainer of this project isnt very active but has merged updates / changes in the recent past, my guess is they will eventually merge 1105, who knows when.

In the meantime, no one else can merge it and this PR, again, is just a duplicate that serves no purpose and should be closed. Extra 'noise'.

@DerekLiang DerekLiang closed this Aug 12, 2024
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.

2 participants