Skip to content

GitHub Action#561

Merged
kprinssu merged 37 commits into
VoodooI2C:2.9from
averycblack:github-action-v2
Nov 3, 2024
Merged

GitHub Action#561
kprinssu merged 37 commits into
VoodooI2C:2.9from
averycblack:github-action-v2

Conversation

@averycblack
Copy link
Copy Markdown
Contributor

@averycblack averycblack commented Nov 1, 2024

Separates out docs and linting to their own scripts and github action jobs. This allows people to build VoodooI2C without running linting/doc building locally.
Automatically uploads kext and docs on every release

Cldoc crashes on an unnamed struct in VoodooI2CHID. This likely means that there are more issues in cldoc on newer revisions, though I haven't really dived deeper into cldoc. I did update it to clang 15.0.7 though that has not helped.

Companion changes:
VoodooI2C/VoodooI2CHID#76
VoodooI2C/cldoc#4

TODO:

  • Get commit history to put changelog into release body
  • Upload docs to github pages

@averycblack averycblack changed the base branch from master to 2.9 November 2, 2024 22:08
@averycblack averycblack marked this pull request as ready for review November 3, 2024 20:45
Copy link
Copy Markdown
Collaborator

@kprinssu kprinssu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! Thank you fixing all the upstream CI issues.

@kprinssu kprinssu merged commit 178808c into VoodooI2C:2.9 Nov 3, 2024
@averycblack averycblack deleted the github-action-v2 branch November 3, 2024 21:06
kprinssu added a commit that referenced this pull request Nov 5, 2024
* Sync VoodooGPIO with AMD support

* Update VoodooGPIO and VoodooI2CHID

* Github Actions CI workflow (#538)

* Create objective-c-xcode.yml

* Update objective-c-xcode.yml

* objective-c-xcode.yml: Update setup tools to fix pyparsing

* objective-c-xcode.yml: try fixing pyparsing directly

* objective-c-xcode.yml: use my fixed cldoc

* objective-c-xcode.yml: manually updating pyparsing was always useless

* Clean up CI/CD (#553)

* Drop CircleCI

* Switch back to VoodooI2C cldoc fork

* Set the pip3 --break-system-packages flag to potentially fix CD/CD

* Run actions on all PRs and branches

* Remove cldoc step

* GitHub Action (#561)

* Create main.yml

* Remove Pip Cache

* Fix Lint in VoodooI2CController.cpp

* Fix VoodooI2CControllerDriver Lint

* Try cursorkind=350

* Only allow one job

* Add separate lint and doc tasks, add archive script

* Fix generate description output, missing dependencies

* Use -r in pip install

* build_docs -> build_doc

* Build VoodooInput for docs

* Release try two

* Test commit

* Test commit 2

* Fix num commits and last_ten_commits

* Set number of max commits

* Fetch tags

* Get tags try 2

* Fix git fetch

* Revert back to using the published trigger

* Save off last tagged commit

* I forgot to save the conflict resolution :(

* Fix VoodooI2CHID submodule commit

* Update to latest master for VoodooI2CHID

* Update cldoc to clang 15

* Update cldoc to VoodooI2C master

* Fix NUM_COMMITS

* Echo release notes

* Make sure to output to

* Account for multiline output

* checkout depth 500

* Multi-line input for release action

* Use softprops release action

* Use file to store changelog

* Upload docs to github pages

* Fix repository name match

* Fix upload always running (oops)

* Add confidence bit for palm rejection (#547)

* Add confidence bit for palm rejection

* Set Confidence bit by default

* Remove logging

* Use constant for palm type

* Update Cldoc repo git link (#562)

* Sync satellites and dependencies

* Update GitHub actions from v3 to v4

* Remove unused extra GitHub pipeline

* Only run push builds when commits are pushed to master

---------

Co-authored-by: Kishor Prins <kprinssu@kishor-tessen.local>
Co-authored-by: newperson1746 <danielcrodriguez2012@gmail.com>
Co-authored-by: Avery Black <avery.c.black@gmail.com>
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.

2 participants