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

Create an automated workflow for updating package.json #64

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

Conversation

Yash-Garg
Copy link
Contributor

@Yash-Garg Yash-Garg commented Nov 14, 2019

It updates the dependencies in package.json on push and creates a Pull Request for it.

Squash and merge if it's fine !

@gotenksIN
Copy link
Contributor

demn pro

@out386
Copy link
Owner

out386 commented Nov 15, 2019

Nice idea, but I prefer manually testing after updating dependencies, and as I didn't write any tests, I'd prefer not to merge this now.

@out386 out386 closed this Nov 15, 2019
@Yash-Garg
Copy link
Contributor Author

@out386 Will try to write tests, let's keep the PR open for other people to help!

@out386 out386 reopened this Nov 16, 2019
@gotenksIN
Copy link
Contributor

necrobumping coz bored af

@out386
Copy link
Owner

out386 commented May 30, 2020

If I merge this now, packages that I've not updated in 2 years will get updated. I half want to see exactly how much will break xD

@gotenksIN
Copy link
Contributor

Lol
That'd be fun

@arghyac35
Copy link

If I merge this now, packages that I've not updated in 2 years will get updated. I half want to see exactly how much will break xD

Hi,

I have updated the dependencies here https://github.com/arghyac35/aria-telegram-mirror-bot/commits/master , two things which broke:

  • uuid package
  • aria2 package

For uuid it was a small fix but for aria2 I need to update the functions accordingly new version.

You can have a look.

@gotenksIN
Copy link
Contributor

wew your fork looks nice
will try it soon ™️

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.

4 participants