#56030 closed task (blessed) (fixed)
Update the Underscore library for 6.1
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.1 | Priority: | normal |
Severity: | normal | Version: | |
Component: | External Libraries | Keywords: | good-first-bug needs-patch add-to-field-guide |
Focuses: | Cc: |
Description
An update is available for the Underscore library. Core currently bundles 1.13.3.
If additional updates become available prior to 6.1, this ticket can be used to apply them.
Changelog: https://underscorejs.org/#changelog.
Previously: #55547 (6.0).
Change History (16)
This ticket was mentioned in PR #2885 on WordPress/wordpress-develop by christinavoudouris.
3 years ago
#1
- Keywords has-patch added
christinavoudouris commented on PR #2885:
3 years ago
#2
Sorry, there's an old branch called update-package. Closing this.
This ticket was mentioned in PR #2886 on WordPress/wordpress-develop by christinavoudouris.
3 years ago
#3
Hey all, long time WordPress user here, first time contributing :)
Trac ticket: https://core.trac.wordpress.org/ticket/56030
This ticket was mentioned in PR #2887 on WordPress/wordpress-develop by dilipbheda.
3 years ago
#4
I have updated the latest library version in the register script as well.
Trac ticket: https://core.trac.wordpress.org/ticket/56030
#5
follow-up:
↓ 6
@
3 years ago
@christinavoudouris,
Your PR looks good, You have missed the updated library version on the register script. I have changed the version and created a new PR.
#6
in reply to:
↑ 5
@
3 years ago
Replying to dilipbheda:
Hey, thanks! I didn't know I was supposed to update that file but now I'm learning. I was confused about the merge conflicts occurring and not sure why a few of my packages are different. Let me know if there's something else I should do on my end.
@christinavoudouris,
Your PR looks good, You have missed the updated library version on the register script. I have changed the version and created a new PR.
christinavoudouris commented on PR #2885:
3 years ago
#7
<s>Sorry, there's an old branch called update-package.</s> Closing this.
#8
@
3 years ago
- Keywords changes-requested added
- Owner set to christinavoudouris
- Status changed from new to assigned
@christinavoudouris @dilipbheda Thanks for these PRs! Looks like this is close.
I was confused about the merge conflicts occurring and not sure why a few of my packages are different.
@christinavoudouris I think the merge conflicts are occurring because your fork is a bit outdated. You should be able to use the fetch and merge option when viewing your fork. Then you can recreate your PR, or pull the changes to your fork's trunk
branch into your pull request branch.
It looks like both PRs change different files, but all of those changes are needed:
package.json
package-lock.json
script-loader.php
I'm assigning the ticket to you @christinavoudouris as that's the way good-first-bug
tickets are marked as claimed.
#9
@
3 years ago
Hmm I thought I had grabbed the latest version. I just went back in and updated the packages in package.json that did not line up. Now it's basically telling me there's still a conflict because I'm going from underscore 1.13.3 to 1.13.4 which is exactly what I'm trying to update anyway. I *did* have underscore 1.13.1 when I initially pulled though. Do we want to go to the latest version?
UPDATE: I made a few more commits and it looks like this is ready to be merged, with the latest version of Underscore. Thank you both for your input!
Is there a way to delete PR 2885?
#10
@
3 years ago
- Keywords commit added; changes-requested removed
Thanks @christinavoudouris! This looks good to go.
#14
@
3 years ago
- Keywords needs-patch added; has-patch commit removed
- Resolution fixed deleted
- Status changed from closed to reopened
While updating other dependencies today, I noticed that releasing a 1.13.5 version is in progress. Re-openeing to follow up once the maintainers finish pushing that out.
Hey all, long time WordPress user here, first time contributing :)
Trac ticket: https://core.trac.wordpress.org/ticket/56030