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

Bump dependencies of dustjs-linked to 3.0 #149

Merged
merged 2 commits into from
Feb 13, 2024

Conversation

slreynolds
Copy link
Contributor

Updated the dependencies of dustjs-linkedin to 3.0.

Fixes issue #148

@gilly3
Copy link

gilly3 commented Feb 9, 2023

Not that I have any actual authority in this repo to approve changes, but if we can get this merged and published, it would save me from forking it and publishing to my company's private registry.

@gilly3
Copy link

gilly3 commented Feb 10, 2023

@jimmyhchan, you most recently merged a PR in this repo. Can you please merge this one (and publish to npm) to address security concerns?

@sumeetkakkar, you most recently merged a PR in the dustjs repo. Any chance you are also authorized to merge this PR?

@sumeetkakkar
Copy link

cc @grawk

@opletin
Copy link

opletin commented Feb 13, 2024

@grawk Matt can you accept it pls?

@grawk grawk merged commit 1469a7a into LinkedInAttic:master Feb 13, 2024
@mikaello
Copy link

Thanks @grawk 🤩 Would it be any chance to publish v1.8.0 to npmjs.com?

@grawk
Copy link
Collaborator

grawk commented Feb 13, 2024

@mikaello Unfortunately I don't have publish permission for the NPM package:

$ npm owner ls dustjs-helpers --registry=https://registry.npmjs.org
rragan <[email protected]>
jimmyhchan <[email protected]>
linkedin <[email protected]>
sethkinast <[email protected]>

Let me plumb the depths of my LinkedIn messages to see if I can find the current holder of permissions.

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.

6 participants