Skip to content
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

Refactor ci #3

Draft
wants to merge 82 commits into
base: develop
Choose a base branch
from
Draft

Refactor ci #3

wants to merge 82 commits into from

Conversation

calvinmorrow
Copy link
Owner

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.

Addresses # (issue)

Dependencies

  • List any dependencies that are required for this change, including a full list of libraries required, especially if it is a new or otherwise unused library in the OpenHornet software.

Type of change

  • New software module (new software module for slave)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update outside of the automatically-generated doxygen documentation.

Checklist:

  • My code follows the (style guidelines)[https://jrsteensen.github.io/OpenHornet-Software/d4/d46/md__2github_2workspace_2_s_t_y_l_e_g_u_i_d_e.html] of this project
  • (I have complied with the software manual)[https://jrsteensen.github.io/OpenHornet-Software/d7/d78/md__software_manual.html] for this project
  • I have performed a self-review of my own code
  • I have commented my code fully with Doxygen compatible comments, particularly in hard-to-understand areas
  • I have made corresponding changes to non-Doxygen generated documentation
  • My changes generate no errors on compile in Arduino IDE
  • My changes generate no new warnings on compile in Arduino IDE
  • Any dependent changes have been merged and published in downstream modules

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • I have tested the sketch in-circuit in DCS with DCS-BIOS and outputs (displays, LEDs, etc.) function as expected.
  • I have tested the sketch in-circuit in DCS with DCS-BIOS and HID inputs (switchs, pots, etc.) function as expected, with switches moving the correct direction.
  • I have tested the sketch in-circuit in DCS with DCS-BIOS and any logic in the sketch has been tested and functions as expected.
  • This code has not yet been tested in-circuit.
  • This code has not yet been tested in DCS-BIOS.

Description

Test Configuration

  • Firmware version:
  • Hardware:
  • Toolchain:
  • SDK:

calvinmorrow and others added 30 commits October 12, 2023 17:18
Added `@details` tag for doxygen header.
Changed license from BY-NC-SA 3.0 to Apache 2 in file header.
Closes jrsteensen#13
Broke Doxygen Workflow into two separate workflows: Main branch workflow generates doxygen and deploys it to gh-page, while develop branch workflow just generates the docs to the ./docs/html folder.
Buuuuut....it helps if the develop branch workflow doesn't deploy to gh-pages...
Non-Release Merge of Develop to Main for testing.
Reverted some changes to include more information in the template and not break doxygen docs.
Fixed `Doxygen` typo.
Moved contributing info to wiki.
Updated repo templates
Updated repo readme
Moved contributing to wiki
Updated sketch template
Integrated Sandra's Master Arm panel code into the example one.

Co-Authored-By: Calvin Morrow <[email protected]>
Co-Authored-By: Sandra Carroll <[email protected]>
Diagnosing why the sketch is not populating in doxygen when the template is.
Got it showing doxygen and both file's info.

(Local testing is important boys and girls!)
Updated doxygen and fixed files to display appropriately.
Arribe and others added 30 commits March 2, 2024 21:49
Updated 1A6-SPIN_RCVY_PANEL.ino with the latest style guide for comments and naming standard.
Updated comments per the latest template for docs.
Added logo to header
Updated title
Updated description
Added version number (which stays the same until release.)
Mostly just still experimenting with header and logo stuff.
Fixed logo link.
Added a couple helpful links
Made spelling corrections throughout. No changes to code itself. (Mostly just testing cLion IDE.)

Signed-off-by: John Steensen <[email protected]>
For general shared IDE configuration as desired moving forward.

Signed-off-by: John Steensen <[email protected]>
* refactored_comments-1A7-HUD_PANEL

Updated comments per the latest template for docs.

* New Sketch 4A7A1-COMM_PANEL

New sketch for the comm panel.

* Update 4A7A1-COMM_PANEL.ino

added attention note on spinning the ILS rotary.
Attribution was misspelled as `Atribution`.

Signed-off-by: John Steensen <[email protected]>
* refactored_comments-1A7-HUD_PANEL

Updated comments per the latest template for docs.

* Feature New Sketch 4A4A2-EXT_LIGHTS_PANEL

New sketch for the 4A4A2-EXT_LIGHTS_PANEL

* Update 4A4A2-EXT_LIGHTS_PANEL.ino

updated comments

* Update 4A4A2-EXT_LIGHTS_PANEL.ino

updated description as reqeusted.
* refactored_comments-1A7-HUD_PANEL

Updated comments per the latest template for docs.

* New Sketch 4A5A2-APU_PANEL

New sketch for the 4A5A2-APU_PANEL with mag switch logic.

* Update 4A5A2-APU_PANEL.ino

fixed code layout for doxygen docs.
* refactored_comments-1A7-HUD_PANEL

Updated comments per the latest template for docs.

* New Sketch 4A5A1-FUEL_PANEL

New sketch for the 4A5A1-FUEL_PANEL, with fuel dump mag-switch logic.
Fixed spelling throughout sketches. No change to programming, version number not incremented.

Signed-off-by: John Steensen <[email protected]>
Fixed spelling throughout sketches. No change to programming, version number not incremented.

Signed-off-by: John Steensen <[email protected]>
* refactored_comments-1A7-HUD_PANEL

Updated comments per the latest template for docs.

* New Sketch 4A6A1-FCS_PANEL

New sketch for the FCS panel.
* refactored_comments-1A7-HUD_PANEL

Updated comments per the latest template for docs.

* New Sketch 4A7A2-OBOGS_PANEL

New sketch for the OBOGS panel.
* refactored_comments-1A7-HUD_PANEL

Updated comments per the latest template for docs.

* New Skectch 5A10-DEFOG_PANEL

New sketch for the 5A10-DEFOG_PANEL with Joystick Library added as a new submodule.
* refactored_comments-1A7-HUD_PANEL

Updated comments per the latest template for docs.

* New Sketch 5A8A1-SIM_CNTL_PANEL

New sketch for the 5A8A1-SIM_CNTL_PANEL.
* refactored_comments-1A7-HUD_PANEL

Updated comments per the latest template for docs.

* New Sketch 5A8A1-SIM_CNTL_PANEL

New sketch for the 5A8A1-SIM_CNTL_PANEL.

* New Sketch 3A2A1-SEAT_CONTROLS

New sketch for the seat.
* refactored_comments-1A7-HUD_PANEL

Updated comments per the latest template for docs.

* New Sketch 5A8A1-SIM_CNTL_PANEL

New sketch for the 5A8A1-SIM_CNTL_PANEL.

* New Sketch 5A9A1-KY58_PANEL.ino

New sketch for the 5A9A1-KY58_PANEL
* refactored_comments-1A7-HUD_PANEL

Updated comments per the latest template for docs.

* New Sketch 5A8A1-SIM_CNTL_PANEL

New sketch for the 5A8A1-SIM_CNTL_PANEL.

* New Sketch 5A6A1-INTR_LT_PANEL.ino

New sketch for the 5A6A1-INTR_LT_PANEL
* refactored_comments-1A7-HUD_PANEL

Updated comments per the latest template for docs.

* New Sketch 5A8A1-SIM_CNTL_PANEL

New sketch for the 5A8A1-SIM_CNTL_PANEL.

* New Sketch 5A7A1-SNSR_PANEL

5A7A1-SNSR_PANEL.ino

* Refactored Sensor Sketch Panel

Updated the code to use similar logic to DcsBios::SwitchMultiPosition to add in debounce logic.  Also created a switch specific class to handle the radar switches pull logic so it can get into and out of EMERG with the wiring as specified in the interconnect diagram.

* Just had ChatGPT comment the header file.

Signed-off-by: John Steensen <[email protected]>

---------

Signed-off-by: John Steensen <[email protected]>
Co-authored-by: John Steensen <[email protected]>
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.

4 participants