<!DOCTYPE html>
<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body>
    Solterra,<br>
    <br>
    sounds great, looking forward to your PR.<br>
    <br>
    Basic requirements for the vehicle code and name spaces are still
    the same as documented in the developer guide. Missing there is only
    the cmake preparation (for IDF5), best to take an existing vehicle
    module as an example.<br>
    <br>
    Check list for the pull request:<br>
    <ul>
      <li>Manual page included and manual building via sphinx without
        errors? ("…/docs/index.rst")<br>
      </li>
      <li>Link to that manual page added to "/README.md"?</li>
      <li>Entry added to "changes.txt"?</li>
      <li>Vehicle build defs added to "support/sdkconfig.default.hw31"?
        (no longer strictly needed I think -- Marc?)<br>
      </li>
    </ul>
    Also, if you need to include some framework changes, create a
    separate PR for those. Generally keep framework changes separate
    from vehicle changes in the git commits as well. Best practice is to
    reduce any commit to a single feature addition/change, and if your
    vehicle code needs a framework change, submit the framework change
    first.<br>
    <br>
    Regards,<br>
    Michael<br>
    <br>
    <br>
    <div class="moz-cite-prefix">Am 25.09.24 um 03:01 schrieb
      solterra--- via OvmsDev:<br>
    </div>
    <blockquote type="cite"
      cite="mid:e5a5b503-5dbb-470c-bd13-9cf8bdc2c944@kezarnet.com">Hi
      all,
      <br>
      <br>
      I'm getting ready to create a pull request to add support for the
      Toyota eTNGA platform vehicles, specifically the Toyota bZ4X and
      Subaru Solterra. I’ve been developing this in my fork for quite
      some time, and I want to make sure the structure of the PR aligns
      with the project’s standards and expectations.
      <br>
      <br>
      Given the length of time this has been in development, I’m seeking
      any advice or guidance on best practices for structuring the pull
      request or specific areas that might need special attention.
      <br>
      <br>
      I’d appreciate any input that can help streamline this process and
      ensure a smooth integration.
      <br>
      <br>
      _______________________________________________
      <br>
      OvmsDev mailing list
      <br>
      <a class="moz-txt-link-abbreviated" href="mailto:OvmsDev@lists.openvehicles.com">OvmsDev@lists.openvehicles.com</a>
      <br>
      <a class="moz-txt-link-freetext" href="http://lists.openvehicles.com/mailman/listinfo/ovmsdev">http://lists.openvehicles.com/mailman/listinfo/ovmsdev</a>
      <br>
    </blockquote>
    <br>
    <pre class="moz-signature" cols="72">-- 
Michael Balzer * Helkenberger Weg 9 * D-58256 Ennepetal
Fon 02333 / 833 5735 * Handy 0176 / 206 989 26</pre>
  </body>
</html>