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

Merge-shared-changes #435

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

Merge-shared-changes #435

wants to merge 19 commits into from

Conversation

millerds
Copy link
Contributor

Change Description:
Merge common changes from taskpane template

  1. Do these changes impact any npm scripts commands (in package.json)? (e.g., running 'npm run start')
    Removes some less used commands.

  2. Do these changes impact VS Code debugging options (launch.json)?
    Changed npm command called

  3. Do these changes impact template output? (e.g., add/remove file, update file location, update file contents)
    No.

  4. Do these changes impact documentation? (e.g., a tutorial on https://docs.microsoft.com/en-us/office/dev/add-ins/overview/office-add-ins)
    Maybe . . . removed some start script commands in favor arguments to the remaining start command

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:
Build and ran automated tests

Rick-Kirkham and others added 18 commits April 25, 2024 13:57
Merge master to yo-office
remove unused manifest property and fix spelling
* Update Fabric Core URL

* Update version
…tart-scripts

Remove extra "start:" scripts from package.json
Bumps [cookie](https://github.com/jshttp/cookie) and [express](https://github.com/expressjs/express). These dependencies needed to be updated together.

Updates `cookie` from 0.6.0 to 0.7.1
- [Release notes](https://github.com/jshttp/cookie/releases)
- [Commits](jshttp/cookie@v0.6.0...v0.7.1)

Updates `express` from 4.21.0 to 4.21.1
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/4.21.1/History.md)
- [Commits](expressjs/express@4.21.0...4.21.1)

---
updated-dependencies:
- dependency-name: cookie
  dependency-type: indirect
- dependency-name: express
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
…eline-template

Update to official pipeline template
@millerds millerds requested a review from a team as a code owner February 21, 2025 23:28
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.

3 participants