fix: remove rules_shell dep

[longlho]

Cleanup and Improve Bazel Module Configuration

TL;DR

Reorganize dependencies, implement a native formatjs_verify_test rule, and improve documentation.

What changed?

• Reorganized Bazel module dependencies:
  • Properly categorized core vs. development dependencies
  • Added descriptive comments for each dependency
  • Removed unused rules_shell dependency
  • Uncommented the compatibility_level = 0 setting
• Replaced the shell-based formatjs_verify_test macro with a native Bazel rule implementation:
  • Removed dependency on rules_shell
  • Improved error handling and test output
  • Enhanced documentation with better examples
• Added links to FormatJS CLI documentation in all rule docstrings
• Cleaned up test comments and improved readability in example code

How to test?

1. Run the example tests to verify the new formatjs_verify_test rule works correctly:

   bazel test //examples/simple/tests_verify/...
   

2. Verify that the module loads correctly without the removed dependencies:

   bazel build //...
   

Why make this change?

• Improved maintainability: Native rule implementation is more robust than shell scripts
• Better developer experience: Enhanced documentation and clearer dependency structure
• Reduced dependencies: Removing rules_shell simplifies the dependency graph
• Consistency: Better organization of the module structure follows Bazel best practices

[longlho]

• feat: allow custom formatjs_cli toolchain #24
• fix: remove rules_shell dep #23 👈 (View in Graphite)
• feat: allow aggregate to output to specific file #22
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[longlho]

Merge activity

• Jan 6, 3:04 PM UTC: @longlho merged this pull request with Graphite.