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

Update grunt. #27

Closed
wants to merge 1 commit into from
Closed

Update grunt. #27

wants to merge 1 commit into from

Conversation

rosiel
Copy link
Contributor

@rosiel rosiel commented Jul 20, 2022

GitHub Issue: Dependabot warnings (3 of them)

What does this Pull Request do?

In package.json, update the grunt version.

This does not solve all the problems, because we have a included a whole darn npm package within this repo, here, that hasn't been updated in a good many years.

What's new?

  • Changes version of grunt.
  • Does this change require documentation to be updated? don't think so.
  • Does this change add any new dependencies? don't think so.
  • Does this change require any other modifications to be made to the repository
    (i.e. Regeneration activity, etc.)? don't think so.
  • Could this change impact execution of existing code? don't think so.

How should this be tested?

  • I do not know, I am not a theme developer.

Additional Notes:

Any additional information that you think would be helpful when reviewing this
PR.

Interested parties

Tag (@ mention) interested parties or, if unsure, @Islandora/committers

@rosiel rosiel closed this Jun 23, 2023
@rosiel rosiel deleted the fix-dependabot branch June 23, 2023 17:01
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.

1 participant