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

docs, tests: update links to graph tooling repo and use latest graph-ts/cli versions #4303

Merged
merged 1 commit into from
May 26, 2023

Conversation

saihaj
Copy link
Member

@saihaj saihaj commented Jan 19, 2023

No description provided.

@saihaj saihaj requested a review from neysofu January 30, 2023 21:50
@neysofu
Copy link
Member

neysofu commented Feb 6, 2023

@saihaj I saw your message on Slack - I've tried to get this thing working, but no luck. It might require us to upgrade Yarn to v2 or v3 to leverage their new workspace commands...

If you're okay with leaving the latest thing out of this PR to revisit it down the road, I think that would be the best course of action.

@saihaj
Copy link
Member Author

saihaj commented Feb 6, 2023

If you're okay with leaving the latest thing out of this PR to revisit it down the road, I think that would be the best course of action.

This issue is we have moved combined graph-cli and graph-ts to monorepo so this is just easier to do

@neysofu
Copy link
Member

neysofu commented Feb 8, 2023

Well, agreed, if we could get it to compile that would be best. Feel free to experiment with it and see if you can get it working.

@azf20
Copy link
Contributor

azf20 commented Mar 20, 2023

hey @neysofu and @saihaj, any update on this, is this blocked or are there changes required here?

@saihaj
Copy link
Member Author

saihaj commented Mar 22, 2023

hey @neysofu and @saihaj, any update on this, is this blocked or are there changes required here?

need to spend sometime debugging this one but haven't gotten to it yet. Putting this in draft for now

@saihaj saihaj marked this pull request as draft March 22, 2023 03:27
@dotansimha dotansimha force-pushed the saihaj/update-graph-tooling-links branch 3 times, most recently from ac07cf6 to 7c48f4a Compare April 18, 2023 09:46
@dotansimha dotansimha force-pushed the saihaj/update-graph-tooling-links branch 2 times, most recently from 35289a6 to 2ac49b2 Compare May 2, 2023 07:50
@dotansimha dotansimha force-pushed the saihaj/update-graph-tooling-links branch 6 times, most recently from 21798b0 to 13d2632 Compare May 24, 2023 08:16
docs: update links for graph tooling repo

dependabot: weekly yarn.lock upgrade

rebuild lockfile

update lockfiles

fixes
@dotansimha dotansimha force-pushed the saihaj/update-graph-tooling-links branch from 13d2632 to 0435b2a Compare May 24, 2023 10:17
@dotansimha dotansimha changed the title docs: update links to graph tooling repo docs, tests: update links to graph tooling repo and use latest graph-ts/cli versions May 24, 2023
@dotansimha dotansimha marked this pull request as ready for review May 24, 2023 11:39
@saihaj saihaj merged commit 08da7cb into master May 26, 2023
@saihaj saihaj deleted the saihaj/update-graph-tooling-links branch May 26, 2023 17:43
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.

5 participants