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 webexnetworkplayer checksum #326

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

Conversation

dosentmatter
Copy link

No description provided.

@AnthonyMastrean
Copy link
Owner

AnthonyMastrean commented Jan 12, 2019

Gotta love WebEx not putting any kind of versioning in their URL :) Is there a version bump to go with this checksum change?

Checksum is for Network Recording Player Version 31.23.4.6
@dosentmatter dosentmatter force-pushed the bugfix/webexnetworkplayer-update branch from e94019f to c2d6334 Compare January 15, 2019 01:26
@dosentmatter
Copy link
Author

dosentmatter commented Jan 15, 2019

Sorry for the delay. The checksum is for version 31.23.4.6. I'm guessing there is a version bump since your previous commit message says 31.14.4
image

I did a force push to my fork to have the version in the commit message.

@AnthonyMastrean
Copy link
Owner

Whoa, sorry for the delay (we had a baby since your posted 😄)! I don't see the version change here with the checksum change. Can you try kicking github? or close and reopen the PR?

@AnthonyMastrean
Copy link
Owner

Would you like to take over maintainership of this package? #392

@dosentmatter
Copy link
Author

Hmm I'm probably not the best maintainer of this package since I haven't been using chocolatey or webex for more than a year now. How does publishing work for your packages? Is it done automatically with CICD or manually? I think the checksums can be automated to some extent.
Either automate publish after a merged MR or regularly poll the download to check ETag and download + update hash if it changed.

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.

2 participants