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

Migrate CI to GitHub Actions #102

Merged
merged 61 commits into from
Apr 16, 2021
Merged

Migrate CI to GitHub Actions #102

merged 61 commits into from
Apr 16, 2021

Conversation

LinqLover
Copy link
Collaborator

@LinqLover LinqLover commented Mar 16, 2021

Closes #101 and closes #97.

This PR:

  • migrates the entire CI pipeline from Travis CI to GitHub Actions, motivated by quota considerations, platform coolness & problems with Slack notifications as described in Add slack notifications for build failures #97:

    image

  • decouples single CI jobs for testing, @test assertions in TEX, and screenshot/PDF generation for a faster and more fine-granular feedback on GitHub (see above)

    • this involves a split-up of the .smalltalk.ston files and related scripts
    • also extracts internal SBE tests into a separate package called SBE-Tests
  • implements support for GitHub Actions CI in Makefile and SmalltalkSources

  • adds Slack notifications if a scheduled build (currently run once a week) fails:

    image

  • provides PDF builds as artifacts on GitHub Actions instead of uploading them to Google Docs or GitHub Releases any longer:

    image

  • fixes the robustize-figures mode (aka DEBUG_FIGURES) which was broken due to an update of the graphicx package and since 85b2080 (thanks again to @AxelKrypton for helping with this issue!)

  • updates the documentation in the README accordingly.

Please squash when merging.

@LinqLover LinqLover self-assigned this Mar 16, 2021
Simple separation of concerns ...
@LinqLover
Copy link
Collaborator Author

Artifacts such as PDF are only accessible when all jobs have been completed, reported here: actions/upload-artifact#181

@LinqLover
Copy link
Collaborator Author

Slack notifications are failing, opened an issue here: voxmedia/github-action-slack-notify-build#46

@LinqLover
Copy link
Collaborator Author

Message from smalltalkCI:

Unable to parse Coveralls build data from /home/runner/.smalltalkCI/_builds/coveralls_build_data.json:
STONReaderError: At character 465: 'invalid escape character \i'
This is a smalltalkCI internal error. If the problem persists, please open an issue at https://github.com/hpi-swa/smalltalkCI/issues/new .

Todo: If this problem persists, let's create an issue.

@LinqLover
Copy link
Collaborator Author

LinqLover commented Mar 17, 2021

Todos before merging:

  • remove all debugging artifacts
  • merge Fix figure Morphic/detachingMenu #103 and make sure to get a green bar afterward
  • check both PDFs for hick-ups and version mismatches
  • squash when merging
  • Update docs
  • after merging: adjust required checks in repository settings

@LinqLover LinqLover marked this pull request as ready for review April 13, 2021 13:56
@LinqLover LinqLover requested a review from codeZeilen April 13, 2021 13:57
@codeZeilen
Copy link
Member

codeZeilen commented Apr 14, 2021

Wow, nice work @LinqLover ! Do you need a detailed review of specific parts or just a general, sweeping look? :)

@LinqLover
Copy link
Collaborator Author

Wow, nice work @LinqLover ! Do you need a detailed review of specific parts or just a general, sweeping look? :)

Sweeping look should be enough :)

@LinqLover LinqLover enabled auto-merge (squash) April 16, 2021 13:19
@LinqLover LinqLover disabled auto-merge April 16, 2021 13:21
@LinqLover LinqLover merged commit 4a12204 into master Apr 16, 2021
@LinqLover LinqLover deleted the github-actions branch April 16, 2021 13:22
LinqLover added a commit that referenced this pull request Apr 27, 2021
* Adds missing docs that should have been part of #102 (4ad7dca)

* Add automated releases to CI again (fixes #106)
codeZeilen pushed a commit that referenced this pull request Feb 28, 2022
Closes #101 and closes #97.

* Migrated the entire CI pipeline from Travis CI to GitHub Actions, motivated by quota considerations, platform coolness & problems with Slack notifications as described in #97

* Decouples single CI jobs for testing, @test assertions in TEX, and screenshot/PDF generation for a faster and more fine-granular feedback on GitHub

  + this involves a split-up of the .smalltalk.ston files and related scripts
  + also extracted internal SBE tests into a separate package called SBE-Tests
  + implemented support for GitHub Actions CI in Makefile and SmalltalkSources

* Added Slack notifications if a scheduled build (currently run once a week) fails

* Provided PDF builds as artifacts on GitHub Actions instead of uploading them to Google Docs or GitHub Releases any longer

* Fixed the robustize-figures mode (aka DEBUG_FIGURES) which was broken due to an update of the graphicx package and since 85b2080 (thanks again to @AxelKrypton for helping with this issue!)

* Updated the documentation in the README accordingly

Co-authored-by: Alessandro Sciarra <[email protected]>
@LinqLover LinqLover mentioned this pull request Jun 27, 2023
3 tasks
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.

Migrate CI to GitHub Actions
3 participants