search

modelica / VehicleInterfaces

Free (standard conforming) library for interface definitions and architectures for vehicle system modeling
Other
43 stars 22 forks source link

Add GitHub-related CI stuff #94

Closed tobolar closed 3 years ago

tobolar commented 3 years ago

Chages of files as compared to those used for MSL:

.CI/
  check_deprecated_line_color.py  . . untouched
  check_html.py . . . . . . . . . . . untouched
  compiler.sh . . . . . . . . . . . . deleted
  Jenkinsfile . . . . . . . . . . . . Library build stuff deleted/commented
  parse.mos . . . . . . . . . . . . . load VehicleInterfaces

.CI/checkHTMLDoc/
  checkLinks.py . . . . . . . . . . . untouched
  checkTags.py  . . . . . . . . . . . untouched
  tidyHTML.py . . . . . . . . . . . . untouched
  __init__.py . . . . . . . . . . . . untouched

.CI/classCoverage/
  CountClasses.mos  . . . . . . . . . load VehicleInterfaces
  CountClasses.py . . . . . . . . . . untouched

.CI\compilers
  Dockerfile  . . . . . . . . . . . . untouched

.CI/Test  . . . . . . . . . . . . . . deleted

.github/workflows
  checkCI.yml . . . . . . . . . . . . use for VehicleInterfaces; Library build stuff deleted/commented
  checkCron.yml . . . . . . . . . . . untouched

I'm especially not sure if set properly .CI/classCoverage/CountClasses.mos and .CI/parse.mos - regarding setModelicaPath, etc.

tobolar commented 3 years ago

Obviously, the suggested CI changes already apply on itself. :-0

CI fails on deprecated Text.lineColor. I have fix this in branch https://github.com/modelica/VehicleInterfaces/tree/replaceDeprecatedLineColor which can be PR-ed as long as the current PR will be merged. (In order to run CI on it.)

beutlich commented 3 years ago

Obviously, the suggested CI changes already apply on itself. :-0

Only the GitHub Actions part, but not the Jenkins part @sjoelund

sjoelund commented 3 years ago

I guess the entire Jenkinsfile could be removed. Do you want the CountClasses script? It could be added as github action if you do want it.

tobolar commented 3 years ago

Do you want the CountClasses script?

I have no clue. Probably not at the moment.

sjoelund commented 3 years ago

Do you want the CountClasses script?

I have no clue. Probably not at the moment.

Then remove the following:

tobolar commented 3 years ago

Seems to be fine now. Any more comment? @sjoelund @beutlich

tobolar commented 3 years ago

@sjoelund @beutlich Thanks a lot!