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

Feature/update guessit #7081

Merged
merged 4 commits into from
Aug 23, 2019
Merged

Feature/update guessit #7081

merged 4 commits into from
Aug 23, 2019

Conversation

p0psicles
Copy link
Contributor

@p0psicles p0psicles commented Aug 22, 2019

  • PR is based on the DEVELOP branch
  • Don't send big changes all at once. Split up big PRs into multiple smaller PRs that are easier to manage and review
  • Read the contribution guide

* Used sharks medusa_vendor_tools.
ext/readme.md Outdated Show resolved Hide resolved
ext/readme.md Outdated Show resolved Hide resolved
ext/readme.md Outdated Show resolved Hide resolved
sharkykh
sharkykh previously approved these changes Aug 22, 2019
requirements.txt Outdated
@@ -1,5 +1,5 @@
https://codeload.github.com/pymedusa/adba/tar.gz/112c04b50d318802b4a0c7924f8e26aaf9e60a5b#egg=adba
https://codeload.github.com/Diaoul/babelfish/tar.gz/f403000dd63092cfaaae80be9f309fd85c7f20c9#egg=babelfish
adba @ https://codeload.github.com/pymedusa/adba/tar.gz/112c04b50d318802b4a0c7924f8e26aaf9e60a5b#egg=adba
Copy link
Contributor Author

@p0psicles p0psicles Aug 22, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sharkykh our build is having issues with the "adba @ ...." Should I revert those changes?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh are you talking about security/snyk?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't realize they couldn't read this format. I'll revert it.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope, I guess it's something else, because it's still failing.

@p0psicles
Copy link
Contributor Author

I'll check again tomorrow. Tnx for the support. Really nice tool!

@p0psicles p0psicles closed this Aug 23, 2019
@p0psicles p0psicles reopened this Aug 23, 2019
@p0psicles
Copy link
Contributor Author

@medariox do you know what's going on?

@medariox
Copy link
Contributor

Snyk has some issues atm. Let's ignore it for now.

@medariox medariox merged commit a44484c into develop Aug 23, 2019
@medariox medariox deleted the feature/update_guessit branch August 23, 2019 21:51
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.

3 participants