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

Translating tutorial-tic-tac-toe.md #80

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

SiroDaves
Copy link

@SiroDaves SiroDaves commented Jan 9, 2025

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

@SiroDaves
Copy link
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).

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