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

Add/GitHub actions #217

Merged
merged 3 commits into from
Jan 19, 2025
Merged

Add/GitHub actions #217

merged 3 commits into from
Jan 19, 2025

Conversation

theavege
Copy link
Contributor

Hello!

I add github-actions and dependabot for github-actions. I hope it will be useful.

Thanks.

@lainz
Copy link
Member

lainz commented Jan 18, 2025

@circular17 for me there's no problem of adding the automation, what do you think?

@lainz lainz merged commit bbdb1a6 into bgrabitmap:master Jan 19, 2025
1 check failed
@theavege theavege deleted the add/github-actions branch January 19, 2025 14:15
@theavege
Copy link
Contributor Author

theavege commented Jan 19, 2025

@lainz FYI

 build from test/test_bcrealnumerickeyboard/test_bcnumerickeyboard.lpi

/home/runner/work/bgracontrols/bgracontrols/test/test_bcrealnumerickeyboard/umain.pas(9,9) Fatal: (10022) Can't find unit BCNumericKeyboard used by umain

@circular17
Copy link
Contributor

circular17 commented Jan 22, 2025

There seem indeed to be some projects that did not pass the tests (continuous integration log).

  • test/test_bcthememanager/test.lpi
  • test/test_bcnumerickeyboard/test_bcnumerickeyboard.lpi
  • test/test_bgrascript/test_bgrascript.lpi
  • test/test_bckeyboard/test_bckeyboard.lpi
  • test/test_bgrapascalscript/test_bgrapascalscript.lpi
  • test/test_bcrealnumerickeyboard/test_bcnumerickeyboard.lpi

I don't know whether it is due to how the job is setup or if the projects actually don't compile.

By the way @theavege, I wonder if it would be easy to provide a list of projects that did not pass in the end of the log. This would avoid having to look through the whole log. What do you think?

@lainz
Copy link
Member

lainz commented Jan 22, 2025

These keyboards are not installed by default. As well the bgrapascalscript needs pascalscript to be installed.

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.

3 participants