Skip to content
This repository has been archived by the owner on Dec 3, 2024. It is now read-only.

Bump tar from 2.2.1 to 2.2.2 in /themes/hello-friend-ng #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dependabot[bot]
Copy link

@dependabot dependabot bot commented on behalf of github May 28, 2019

Bumps tar from 2.2.1 to 2.2.2.

Commits
  • 523c5c7 2.2.2
  • 7ecef07 Bump fstream to fix hardlink overwriting vulnerability
  • 9fc84b9 Use {} for hardlink tracking instead of []
  • 15e59f1 Only track previously seen hardlinks
  • 4f85851 Ignore potentially unsafe files
  • See full diff in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot ignore this [patch|minor|major] version will close this PR and stop Dependabot creating any more for this minor/major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label May 28, 2019
@cb-technologists-bot
Copy link
Contributor

Hi @dependabot[bot]. Thanks for your PR.

I'm waiting for a cb-technologists member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@logandonley
Copy link
Contributor

/ok-to-test

@cb-technologists-bot
Copy link
Contributor

⭐ PR built and available in a preview environment cb-technologists-blog-pr-14 here

@logandonley
Copy link
Contributor

/approve

@dcanadillas
Copy link
Contributor

/lgtm

@cb-technologists-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dcanadillas, logandonley
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: kmadel

If they are not already assigned, you can assign the PR to them by writing /assign @kmadel in a comment when ready.

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@cvega
Copy link
Member

cvega commented May 30, 2019

Does it make sense to add this theme as a submodule and commit the change upstream? Maybe we've already deviated from their master or plan to make changes in the future?

@kmadel
Copy link
Contributor

kmadel commented May 30, 2019

@cvega Agreed, and it was a submodule originally but that didn't play nicely with the way the Jenkins X did a deploy for the jx-docs repo - https://github.com/jenkins-x/jx-docs/tree/master/themes/gohugoioTheme - so i took the easy way out and copied that - but I'm sure we could figure out how to tweak the JX Pipeline to deploy with a submodule - at this point I'm not sure any of the minor tweaks would be worth sharing upstream but I'm sure there will be some in the future that would be worth sharing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Pull requests that update a dependency file lgtm ok-to-test size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants