-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[$125] Improve developer experience around react-compiler #48066
Comments
Triggered auto assignment to @mallenexpensify ( |
Current assignee @mallenexpensify is eligible for the NewFeature assigner, not assigning anyone new. |
|
Triggered auto assignment to Design team member for new feature review - @shawnborton ( |
Job added to Upwork: https://www.upwork.com/jobs/~014befe13a62da3c50 |
Current assignee @rayane-djouah is eligible for the External assigner, not assigning anyone new. |
Don't think we need ya here @shawnborton , sorry. I added |
@mallenexpensify The reward can be reduced to $125 according to the process, as PR doesn't require app testing. |
Upwork job price has been updated to $125 |
@rayane-djouah can you please accept the job and reply here once you have? |
@mallenexpensify Accepted! Thanks! |
Contributor+: @rayane-djouah paid $125 via Upwork Don't think a regression test is needed here, plz comment if anyone disagrees and I'll reopen. And/or, @Kicu , if documentation needs to be updated anywhere. Thx |
Version Number:
Reproducible in staging?: -
Reproducible in production?: -
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
This is purely a developer experience issue.
I have recently spent 1+ hours trying to fix an issue with react-compiler on one of my PRs. I think this may happen in future for more people and I wanted to add some tweaks to our code to improve this experience.
Here's what I would like to do:
npm run react-compiler-healthcheck &> output.txt
(npm run react-compiler-healthcheck &> output.txt)output.txt
we would call this file something specific, for examplereact-compiler-output.txt
react-compiler-output.txt
to gitignore, so that a developer can easily keep running the same script and not worry about accidentally pushing it to upstream repo (which I have - of course - did by mistake 😅)I can have a PR ready very soon with these
CC @kirillzyusko
Action Performed:
N/A
Expected Result:
N/A
Actual Result:
N/A
Workaround:
N/A
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @The text was updated successfully, but these errors were encountered: