-
Notifications
You must be signed in to change notification settings - Fork 760
fix: use context.issue in notify translators #4587
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@akshayka Maybe a semi-superifical change to the readme to test it? |
@dmadisetti good call, done |
README Translation Update NeededThis PR includes changes to the main README.md. The following translation files may need to be updated:
Translation maintainers, please review the changes in this PR and update your respective README translations accordingly. CC: @metaboulie @kiwamizamurai @Francisco-G-P |
|
🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.12.11-dev12 |
|
CC: @metaboulie @kiwamizamurai @Francisco-G-P We've set up a workflow that will ping owners for translated READMEs whenever the English README is updated. This PR doesn't actually update the README in a meaningful way, but the translated READMEs have drifted from the English one over time. If you all have the time to revise your READMEs, that would be very much appreciated :) Thanks so much! |
|
Thank you. I'll update it. This is getting popular in Japan |
…ty (#4606) ## 📝 Summary <!-- Provide a concise summary of what this pull request is addressing. If this PR fixes any issues, list them here by number (e.g., Fixes #123). --> I update README for Japanese ## 🔍 Description of Changes <!-- Detail the specific changes made in this pull request. Explain the problem addressed and how it was resolved. If applicable, provide before and after comparisons, screenshots, or any relevant details to help reviewers understand the changes easily. --> Based on the release notification, I modified the documentation #4587 (comment) ## 📋 Checklist - [x] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [ ] I have run the code and verified that it works as expected. ## 📜 Reviewers <!-- Tag potential reviewers from the community or maintainers who might be interested in reviewing this pull request. Your PR will be reviewed more quickly if you can figure out the right person to tag with @ --> @akshayka OR @mscolnick
The previous workflow used an API that doesn't exist on merged pull requests (
github.event)