Skip to content
This repository has been archived by the owner on Sep 27, 2023. It is now read-only.

Upgrade to Relay v2.0.0 #85 #87

Merged
merged 2 commits into from
Jan 29, 2019
Merged

Upgrade to Relay v2.0.0 #85 #87

merged 2 commits into from
Jan 29, 2019

Conversation

artola
Copy link
Contributor

@artola artola commented Jan 28, 2019

  • update dependencies accordingly Relay v2.0.0
  • removed unused dependency @types/immutable
  • added minimalist types for RelayCompilerPublic
  • updated fixtures
  • updated test TypeScriptGenerator to support directives as v2.0.0

artola and others added 2 commits January 28, 2019 01:36
  - update dependencies accordingly Relay v2.0.0
  - removed unused dependency @types/immutable
  - added minimalist types for RelayCompilerPublic
  - updated fixtures
  - updated test TypeScriptGenerator to support directives as v2.0.0
@sibelius sibelius merged commit 177489c into relay-tools:master Jan 29, 2019
@sibelius
Copy link
Contributor

not sure how to update your branch

I've tested here https://github.com/entria/entria-fullstack, and it is working fine

I think we should automate releases based on this https://github.com/intuit/auto

@artola
Copy link
Contributor Author

artola commented Jan 29, 2019

@sibelius Great!

@sibelius @alloy Could we proceed with #86 releasing manually a v2.0 package? As there is a breaking change that is related to relay-compiler v1 changing to v2 (and other dependencies version related), it makes sense to go for v2 too.
People that is wanting to upgrade Relay depends on this package for easy integration.

@alloy
Copy link
Member

alloy commented Jan 29, 2019

Agreed that we should have automatic releases, but for now I'll manually cut a release 👍

@artola
Copy link
Contributor Author

artola commented Jan 29, 2019

@alloy I agree with automatic releases too, just that it should not delay a new release as people is willing to move into Relay v2 asap.
Thanks for your help.

@alloy
Copy link
Member

alloy commented Jan 29, 2019

Done, thanks for all the help! 👌

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants