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

fix: Support Nexus 1.0.0 (#182) #183

Closed
wants to merge 1 commit into from

Conversation

lewebsimple
Copy link
Contributor

I updated the package name and changed RootValue to SourceValue and all tests are passing !

Could it be this simple ?!

@lewebsimple
Copy link
Contributor Author

It seems to work just fine in one of my projects that was updated to nexus 1.0.0 !

@Sytten
Copy link
Owner

Sytten commented Dec 15, 2020

Thanks for that, I will pull it locally just to make sure. I think it might warrant a minor version upgrade instead of a fix.
Should be merged by the end of the day.

@Sytten Sytten linked an issue Dec 15, 2020 that may be closed by this pull request
@lewebsimple
Copy link
Contributor Author

@Sytten did you want to release this with a minor version bump or are you still waiting for other features / bugfixes ?

@Sytten
Copy link
Owner

Sytten commented Dec 22, 2020

Sorry for the delay I had a rough week and enjoying some time off. I will finish everything tomorrow.

@Sytten
Copy link
Owner

Sytten commented Dec 24, 2020

Fixed by #190

@Sytten Sytten closed this Dec 24, 2020
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.

Support nexus 1.0.0
2 participants