Skip to content

Upgrade UUID to 14.0#1002

Open
timwan10 wants to merge 2 commits into
masterfrom
user/timwan/BumpUUID
Open

Upgrade UUID to 14.0#1002
timwan10 wants to merge 2 commits into
masterfrom
user/timwan/BumpUUID

Conversation

@timwan10

@timwan10 timwan10 commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Thank you for your pull request!

Please add '[major]', '[minor]', or [patch] to the title to indicate the impact the change has on the code. Please also provide the following information.


Change Description:

Describe what the PR is about. 
  1. Do these changes impact command syntax of any of the packages? (e.g., add/remove command, add/remove a command parameter, or update required parameters)
    If Yes, briefly describe what is impacted.

  2. Do these changes impact documentation? (e.g., a tutorial on https://learn.microsoft.com/office/dev/add-ins/overview/office-add-ins)
    If Yes, briefly describe what is impacted.

If you answered yes to any of these please do the following:
> Include 'Rick-Kirkham' in the review
> Make sure the README file is correct

Validation/testing performed:

npm run test passes.

@timwan10 timwan10 requested a review from a team as a code owner June 20, 2026 00:32
Comment thread package-lock.json Outdated
"version": "11.0.0",
"resolved": "https://registry.npmjs.org/@types/uuid/-/uuid-11.0.0.tgz",
"integrity": "sha512-HVyk8nj2m+jcFRNazzqyVKiZezyhDKrGUA3jlEcg/nZ6Ms+qHwocba1Y/AaVaznJTAM9xpdFSh+ptbNrhOGvZA==",
"deprecated": "This is a stub types definition. uuid provides its own type definitions, so you do not need this installed.",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This says that it's not necessary anymore, but when I removed it, npm install put it right back... ?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This says that it's not necessary anymore, but when I removed it, npm install put it right back... ?

What do you mean "I removed it" Looks like this is referring to the @types/uuid package and the message is saying it's included in the uuid package directly and so the separate @types package in not needed (i.e. remove it from package.json). This is probably why the version diverged (I believe the types version was in sync with the main package up until version 11 of the types package).

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