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

(#5)(#6) choco-astro 0.1.1 #8

Merged
merged 2 commits into from
Oct 2, 2024
Merged

(#5)(#6) choco-astro 0.1.1 #8

merged 2 commits into from
Oct 2, 2024

Conversation

st3phhays
Copy link
Member

@st3phhays st3phhays commented Sep 25, 2024

Description Of Changes

  • Upgrades all dependencies to the latest versions
  • Updates yarn npm audit command to scan all dependencies recursively

Motivation and Context

We need to ship the latest dependencies to remove any potential vulnerabilities. We also want to stay up to date with the latest Astro framework to take advantage of new features.

Testing

On the docs repository, follow these testing instructions: chocolatey/choco-theme#420

Operating Systems Testing

Tested in conjunction with choco-theme 0.8.1 on Windows 10.

Change Types Made

  • Bug fix (non-breaking change).
  • Feature / Enhancement (non-breaking change).
  • Breaking change (fix or feature that could cause existing functionality to change).
  • Documentation changes.
  • PowerShell code changes.

Change Checklist

  • Requires a change to the documentation.
  • Documentation has been updated.
  • Tests to cover my changes, have been added.
  • All new and existing tests passed?
  • PowerShell code changes: PowerShell v3 compatibility checked?

Related Issue

Fixes #5
Fixes #6

This updates the yarn npm audit command to include all severity
types and scan dependencies recursively.
@st3phhays st3phhays mentioned this pull request Sep 25, 2024
10 tasks
@st3phhays
Copy link
Member Author

This is in draft due to this needing tested on repositories that will use it (docs). Once that separate PR has been reviewed and approved, this can be taken out of draft and merged.

@st3phhays st3phhays self-assigned this Sep 25, 2024
@st3phhays
Copy link
Member Author

Version needs bumped to 0.1.1 in package.json

Copy link
Contributor

@alexaveldanez alexaveldanez left a comment

Choose a reason for hiding this comment

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

Looks good to me.

This bumps all dependencies used in this package to their latest
versions. The Astro specific packages were upgraded by using the
recommended upgrade command in the readme. Changes were then verified
by running the docs site and ensuring no broken functionality.
@st3phhays st3phhays marked this pull request as ready for review October 2, 2024 18:13
Copy link
Contributor

@alexaveldanez alexaveldanez left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@alexaveldanez alexaveldanez merged commit 2b071c8 into main Oct 2, 2024
1 check passed
@st3phhays st3phhays deleted the release/0.1.1 branch October 23, 2024 10:24
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.

Scan for vulnerabilities on all dependencies recursively Upgrade to the latest version of Astro
2 participants