Skip to content

Conversation

@SiroDevs
Copy link
Contributor

@SiroDevs SiroDevs commented Jan 9, 2025

Translating tutorial-tic-tac-toe.md to SW as per issue #1

@SiroDevs
Copy link
Contributor Author

Happy new year @orama254 @katungi @bmwasonga.
As you review help me fix the issue that caused the fail hoping it was not from my end.

@orama254
Copy link
Member

Happy new year @orama254 @katungi @bmwasonga. As you review help me fix the issue that caused the fail hoping it was not from my end.

Hello and happy new year @SiroDaves

I'll make a review and investigate the origin of the failing checks then circle back where necessary.

Thanks

@katungi
Copy link

katungi commented Jan 10, 2025

Hey @orama254 and @SiroDaves , I have identified the issue being one of the workflows using an old version of the artifacts actions. So we need to update that to the latest one.
I can take a crack at it if you guys want, unless Rama has started working on it.

@orama254
Copy link
Member

Hey @orama254 and @SiroDaves , I have identified the issue being one of the workflows using an old version of the artifacts actions. So we need to update that to the latest one. I can take a crack at it if you guys want, unless Rama has started working on it.

@katungi My investigations also yielded the same, my approach was to start merging the sync PRs chronologically until the latest one as there might have been an update on the upstream repo introducing the breaking change. Happy to take it up (or split it into manageable chunks).

@orama254 orama254 merged commit 1f3d9f9 into reactjs:main Mar 6, 2025
1 of 3 checks passed
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.

3 participants