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

Support for 1.18 generics #107

Merged
merged 4 commits into from May 9, 2022
Merged

Support for 1.18 generics #107

merged 4 commits into from May 9, 2022

Conversation

ImVexed
Copy link
Contributor

@ImVexed ImVexed commented Jan 15, 2022

Hello!

I found myself beginning to port some personal projects to 1.18 and found that this library didn't yet have any support for generics. So, I figured I might as well take a swing at it.

I'm not exactly sure what the right path is for making this available is since 1.18 is still a beta. Regardless, hopefully this is useful to others :)

@orcaman
Copy link
Owner

orcaman commented Jan 16, 2022

This is very cool. I am looking forward to reviewing this soon

@chbomtempo
Copy link

@ImVexed You missed the .travis.yml file.

@ImVexed
Copy link
Contributor Author

ImVexed commented Mar 4, 2022

@ImVexed You missed the .travis.yml file.

Not particularly sure what's failing now or why.

@chbomtempo
Copy link

chbomtempo commented Mar 4, 2022

@ImVexed Even though you're using a beta version, seems to be the same issue as with the 1.18rc1... Seems that golangci-lint have some issues with it... google/go-github#2296

@soulmachine
Copy link

any updates? Looking forward to generics support

@ImVexed
Copy link
Contributor Author

ImVexed commented Apr 5, 2022

@chbomtempo seems to be working fine now that 1.18 is out

@orcaman orcaman merged commit b1f44ce into orcaman:master May 9, 2022
@orcaman
Copy link
Owner

orcaman commented May 9, 2022

Thanks @ImVexed , you rock.

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.

None yet

4 participants