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

Build failure #3455

Merged
Merged

Conversation

MVarshini
Copy link
Contributor

@MVarshini MVarshini commented Jun 7, 2023

Build failure

'startTransition' is not exported from 'react'

Workaround to fix the issue

react-router-dom accidentally pushed a breaking change in 6.12. See remix-run/react-router#10566.

'startTransition' is not exported from 'react'
@MVarshini MVarshini added the Dashboard Of and relating to the Dashboard GUI label Jun 7, 2023
@MVarshini MVarshini self-assigned this Jun 7, 2023
Copy link
Member

@dbutenhof dbutenhof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll leave Jenkins as the final judge, but this looks good.

Unfortunately, for every workaround to these package problems, we really need to remember to undo the version lock later after it's fixed. But given it hasn't been fixed yet, this allows us to move forward.

@dbutenhof dbutenhof merged commit c810bf1 into distributed-system-analysis:main Jun 7, 2023
@webbnh
Copy link
Member

webbnh commented Jun 7, 2023

Unfortunately, this change locked us to 6.3.0, rather than to 6.11.2, which is what we've presumably been using (successfully) for the past three weeks. Hopefully this difference doesn't matter, and hopefully this change will only be in effect for a short time....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dashboard Of and relating to the Dashboard GUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants