Skip to content

Conversation

@Mbeaulne
Copy link
Collaborator

@Mbeaulne Mbeaulne commented Dec 9, 2025

Description

This PR enables React Compiler for additional directories and simplifies event handlers in route components:

  1. Added src/components/shared/Authentication and src/routes to the enabled directories in react-compiler.config.js
  2. Removed comments and reorganized the directory list for better readability
  3. Simplified event handlers in ErrorPage.tsx and NotFoundPage.tsx by removing unnecessary useCallback hooks
  4. Fixed formatting in RunRow.tsx component

Type of Change

  • Improvement
  • Cleanup/Refactor

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Test Instructions

  1. Verify that the application functions correctly with React Compiler enabled for the additional directories
  2. Confirm that navigation and error handling work as expected in the routes components

@Mbeaulne Mbeaulne force-pushed the 12-09-remove_routes_for_compiler branch from 1cb82bc to d3e8c0a Compare December 10, 2025 23:30
Copy link
Collaborator Author

Mbeaulne commented Dec 10, 2025

Merge activity

  • Dec 10, 11:31 PM UTC: Graphite rebased this pull request as part of a merge.
  • Dec 10, 11:32 PM UTC: @Mbeaulne merged this pull request with Graphite.

@Mbeaulne Mbeaulne merged commit 3a98a7c into master Dec 10, 2025
8 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