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

Upgrade to OTP 24. #104

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

Conversation

ankhers
Copy link

@ankhers ankhers commented Jun 17, 2021

Erlang's crypto module has removed some deprecated functions in favour
of a new API. This removes uses of the old functions which no longer
work in OTP 24.

Erlang's crypto module has removed some deprecated functions in favour
of a new API. This removes uses of the old functions which no longer
work in OTP 24.
@zdeneksejcek
Copy link

Please merge this one, otherwise we can upgrade to OTP24 and later :(

@Ninigi
Copy link
Collaborator

Ninigi commented Sep 16, 2021

@zdeneksejcek really sorry - we decided to not maintain this package anymore.

The very nature of Shopify updating their API so rapidly makes it extremely hard to keep up with updates. We would have to maintain multiple branches every time to keep up with them.

I would recommend using ankhers fork for the project you are working on, if you know it works for you, or even better, create your own fork. As it is right now, you can use this repo as a blueprint, but it will fall out more out of date every 3 months.

EDIT: Also, we could not just make older OTP version obsolete - so we would have to maintain branches for the Shopify API versions, multiplied by the OTP versions we want to support. If this was an official Shopify package, I might think about it - but like this - open source, and basically only one person ever working on it - it's just too much, I am really sorry.

@ankhers
Copy link
Author

ankhers commented Sep 16, 2021 via email

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