search

modelica / www.modelica.org

The Modelica Association/Language/Library website
https://modelica.org
2 stars 25 forks source link

Remove Impact workflow from CI #67

Closed t-sommer closed 7 months ago

t-sommer commented 7 months ago

since repository has been archived

dietmarw commented 7 months ago

I dont think you should have removed the listing of the libraries. Its only the impact search page and impact tool that is archived. The listing on modelica.org is completely i dependent. The 3rd party repositories continue to exist. So you really should role the merge of this u reviewed pr back again.

ons. 14. feb. 2024, 17:20 skrev Torsten Sommer @.***>:

Merged #67 https://github.com/modelica/www.modelica.org/pull/67 into main.

— Reply to this email directly, view it on GitHub https://github.com/modelica/www.modelica.org/pull/67#event-11804150033, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAACI5HHW7TI7TAUJE26UP3YTTP3HAVCNFSM6AAAAABDISGK6WVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRHAYDIMJVGAYDGMY . You are receiving this because you are subscribed to this thread.Message ID: @.***>

t-sommer commented 7 months ago

We just discussed this in the MA Board Meeting and decided to move this to modelica.org website (see also #66).

dietmarw commented 7 months ago

It is already in the modelica website. See also my comment on #66. Again nothing to do with impact. So really you should not remove the listing until you have a newer one which is working.

dietmarw commented 7 months ago

So please role this PR back until #66 is resolved with an updated view if you want. Also It is strange that PRs are merged without any review for something so visible to the outside world like the website. @hubertus65 ?