jsoref opened a new pull request, #38896:
URL: https://github.com/apache/arrow/pull/38896

   ### Rationale for this change
   
   If you make changes to any of the code generators (src/idl_gen*) be sure to
   
[build](https://google.github.io/flatbuffers/flatbuffers_guide_building.html) 
your project, as it will generate code based on the changes. If necessary
   the code generation script can be directly run (scripts/generate_code.py),
   requires Python3. This allows us to better see the effect of the PR.
   
   I've made changes but haven't actually tried building it, sorry. I can do 
that later...
   
   If your PR includes C++ code, please adhere to the
   [Google C++ Style Guide](https://google.github.io/styleguide/cppguide.html),
   and don't forget we try to support older compilers (e.g. VS2010, GCC 4.6.3),
   so only some C++11 support is available.
   
   Technically yes, but I haven't changed anything meaningful.
   
   Fixes misspellings identified by the [check-spelling 
action](https://github.com/marketplace/actions/check-spelling).
   
   The misspellings have been reported at 
https://github.com/jsoref/apache-arrow/actions/runs/7008141484/attempts/1#summary-19063787889
   
   ### What changes are included in this PR?
   
   ### Are these changes tested?
   
   The action will report that the changes in this PR would make it happy: 
https://github.com/jsoref/apache-arrow/actions/runs/7008142287
   
   ### Are there any user-facing changes?
   
   I hope not -- I'm happy to split any changes into a distinct PR if requested


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to