Skip to content

CI using GitHub workflow - #53

Merged
casella merged 1 commit into
OpenModelica:masterfrom
AnHeuermann:github-action
Jan 22, 2024
Merged

CI using GitHub workflow#53
casella merged 1 commit into
OpenModelica:masterfrom
AnHeuermann:github-action

Conversation

@AnHeuermann

Copy link
Copy Markdown
Member

Test the OpenModelicaLibraryTesting Python scripts by running them on the PNlib (v2.2.0) with OpenModelica nightly and stable.

Resulting HTML files and sqlite are saved as artifacts. After downloading the HTML files can be viewed with

unzip nightly-PNlib.html -d html
python3 -m http.server -d html

@AnHeuermann

Copy link
Copy Markdown
Member Author

@adrpo I think I already found an issue in the generated HTML done by https://github.com/OpenModelica/OpenModelicaLibraryTesting/blob/master/test.py#L825 generating:

<table><tr><th>Commit</th><th>Date</th><th>Author</th><th>Summary</th></tr><tr><td><a href="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/OpenModelica/OpenModelicaLibraryTesting/commit//f212828">f212828</a></td><td>2024-01-16 11:47:45 +0100</td><td>AnHeuermann</td><td>Unique names for artifacts</td></tr></table>

grafik

Or is it an issue with my test script?

@AnHeuermann
AnHeuermann marked this pull request as draft January 16, 2024 11:39
  - GitHub workflow to run scripts with sanityCheck.json
  - Test on Ubuntu with Python 3.10 and omc nightly/stable
  - Resulting HTML and sqlite are archived
@AnHeuermann
AnHeuermann marked this pull request as ready for review January 16, 2024 13:16
@AnHeuermann

Copy link
Copy Markdown
Member Author

On my fork this branch is running and passing the workflow, see https://github.com/AnHeuermann/OpenModelicaLibraryTesting/actions/runs/7542150646/job/20530331899.

One run takes about 6 minutes, we could add a tiny test library to the .github/ directory, but I didn't want to add unnecessary stuff.

@casella
casella merged commit e188a42 into OpenModelica:master Jan 22, 2024
@AnHeuermann
AnHeuermann deleted the github-action branch January 22, 2024 13:42
@AnHeuermann AnHeuermann mentioned this pull request Feb 9, 2024
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