Skip to content

Modern Refactor of lib#4

Open
jeichorn wants to merge 4 commits intoGAuthify:masterfrom
jeichorn:master
Open

Modern Refactor of lib#4
jeichorn wants to merge 4 commits intoGAuthify:masterfrom
jeichorn:master

Conversation

@jeichorn
Copy link
Copy Markdown

This isn't a real pull request, since it would break all backwards compatibility with your current users, so couldn't be merged in this form.

If you are interested in headed in this direction for your official release let me know, otherwise I will maintain this fork since we needed a version of the lib that meets our coding standards.

@pnegahdar
Copy link
Copy Markdown
Member

Great pull request, thanks at @jeichorn. We're releasing v2 of the GAuthify API (which very slightly breaks backwards compatibility) in a week or so so I'll adapt this for that version. The upgrade path on your end should be pretty straight forward and the first version of the API is not being deprecated any time soon.

@baohx2000
Copy link
Copy Markdown

Any possibility of this going in? Looks like v2 never happened.

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