tardis-sn / carsus

Atomic Database for Astronomy
https://tardis-sn.github.io/carsus
21 stars 44 forks source link

Update docs workflow #375

Closed atharva-2001 closed 1 year ago

atharva-2001 commented 1 year ago

:pencil: Description

Type: :beetle: bugfix | :roller_coaster: infrastructure

Docs workflow doesn't get triggered in PRs. For example- https://github.com/tardis-sn/carsus/pull/374

:pushpin: Resources

Examples, notebooks, and links to useful references.

:vertical_traffic_light: Testing

How did you test these changes?

:ballot_box_with_check: Checklist

Note: If you are not allowed to perform any of these actions, ping (@) a contributor.

epassaro commented 1 year ago

I'm quite sure this is not going to work.

andrewfullard commented 1 year ago

Following Ezequiel's comment, my opinion is that we should have as many of the TARDIS workflows apply to Carsus as possible so that they work in a consistent manner across both codebases,

atharva-2001 commented 1 year ago

Would copy workflow from TARDIS instead of this one.