Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Use Github workflows to compile examples and spell check #34

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

Conversation

Copy link
Contributor

@sandeepmistry sandeepmistry commented Oct 2, 2019

Alternative to PR #31 and #32.

Uses compile-examples action from: https://github.com/arduino-libraries/actions

This will allow us to have a super minimal .yml file in all arduino-libraries repos, and re-use a common action from a central Github repo.

Copy link
Contributor Author

@facchinm @cmaglie @lxrobotics @mastrolinux @rsora please review this along with https://github.com/arduino-libraries/actions/tree/master/compile-examples (which is based on https://help.github.com/en/articles/creating-a-docker-container-action).

@rsora was in favour of use Node.js instead of Docker + bash for the common Github action in terms of testability, but I think this is good to get us going for now. We can introduce Node.js down the road as needed.

rsora and cmaglie reacted with thumbs up emoji cmaglie reacted with rocket emoji

Copy link
Contributor

👍 I like how clean everything is. Doesn't need a rocket scientist to comprehend what's happening. Nice work 😉

Copy link

rsora commented Oct 3, 2019
edited
Loading

Please do not let bash proliferate 😸

Anyway, the action properly hides the boilerplate so 👍 from me.

May I suggest to move the https://github.com/arduino-libraries/actions/tree/master/compile-examples code to https://github.com/arduino/actions?
This way we will have a single consistent point of access for all our actions in the future (good also for us maintainers and users).

@sandeepmistry sandeepmistry force-pushed the arduino-libraries-github-action branch 5 times, most recently from 58d1edd to e3d07c8 Compare October 3, 2019 19:47
Copy link
Contributor Author

I've added spell check support In: https://github.com/arduino-libraries/actions/tree/master/spell-check (it uses code spell) ... this works and I've fixed some spelling :)

May I suggest to move the https://github.com/arduino-libraries/actions/tree/master/compile-examples code to https://github.com/arduino/actions?

@rsora I think I'm ok with this, we might need to rename the action names to indicate they are for Arduino Libraries only, with we get for free sort of because of arduino-libraries/actions

Copy link

rsora commented Oct 4, 2019

@rsora I think I'm ok with this, we might need to rename the action names to indicate they are for Arduino Libraries only, with we get for free sort of because of arduino-libraries/actions

@sandeepmistry as per https://help.github.com/en/articles/workflow-syntax-for-github-actions#example-using-a-public-action-in-a-subdirectory we should be able to do something like this
uses: arduino/actions/libraries/compile-examples@master WDYT?

sandeepmistry reacted with heart emoji

Copy link
Contributor Author

sandeepmistry commented Oct 4, 2019
edited
Loading

uses: arduino/actions/libraries/compile-examples@master WDYT?

@rsora excellent suggestion! Stay tuned for a pull request in that repo! #hacktoberfest

Copy link
Contributor Author

This should be good to go now, I've removed the Travis CI config as well. We need to re-add the badges to the read me once merged.

@sandeepmistry sandeepmistry changed the title (削除) [DO NOT MERGE] Compile library examples via Github actions (削除ここまで) (追記) @sandeepmistry Use Github workflows to compile examples and spell check (追記ここまで) Oct 4, 2019
@sandeepmistry sandeepmistry changed the title (削除) @sandeepmistry Use Github workflows to compile examples and spell check (削除ここまで) (追記) Use Github workflows to compile examples and spell check (追記ここまで) Oct 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cmaglie cmaglie Awaiting requested review from cmaglie

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /