-
Notifications
You must be signed in to change notification settings - Fork 46
Update links to graphql client repository #112
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
Update links to graphql client repository #112
Conversation
ad26c50 to
7a44107
Compare
Generated by 🚫 Danger |
yuki24
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had a question about the CHANGELOG entry, but overall 👍
CHANGELOG.md
Outdated
| @@ -1,5 +1,6 @@ | |||
| ### (Next) | |||
| * Your contribution here. | |||
| * Update graphql-client github repository links in README - [@th1988] | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's fine, but we do need to fix CI even if this is unrealated.
dblock
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make the CHANGELOG line like the others, with link to PR and your name, thanks!
We do need to fix CI, I know it's not failing because of this change, but I'd prefer we don't merge anything before that. Maybe you can help?
|
Rebase this please? |
7a44107 to
bde6d9a
Compare
|
Done @dblock 🚀 |
No description provided.