Skip to content

Refactor Badges#310

Open
teutoburg wants to merge 5 commits intodev_masterfrom
fh/badges2
Open

Refactor Badges#310
teutoburg wants to merge 5 commits intodev_masterfrom
fh/badges2

Conversation

@teutoburg
Copy link
Contributor

@teutoburg teutoburg commented Feb 5, 2026

  • Use Badges from astar-utils
  • Delete double implementation of Badges
  • Run internal functionality tests (unrelated to any specific instrument package) separately

This was pulled from a 6 months old branch...

Those don't need to run in the full test matrix.
This has been moved to astar-utils for the longest time, but never actually
update the code to use that...
@teutoburg teutoburg self-assigned this Feb 5, 2026
@teutoburg teutoburg added the irdb functionality Publish script, packaging, badges etc. label Feb 5, 2026
@teutoburg teutoburg moved this to 🏗 In progress in ScopeSim-development Feb 5, 2026
@teutoburg teutoburg changed the title Fh/badges2 Refactor Badges Feb 5, 2026
@teutoburg teutoburg marked this pull request as ready for review February 5, 2026 14:40
@teutoburg
Copy link
Contributor Author

Separate tests are correctly triggered by label: https://github.com/AstarVienna/irdb/actions/runs/21715756177/job/62631118699?pr=310

@teutoburg teutoburg moved this from 🏗 In progress to 👀 Awaiting Review in ScopeSim-development Feb 5, 2026
@teutoburg teutoburg requested a review from a team February 5, 2026 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

irdb functionality Publish script, packaging, badges etc.

Projects

Status: 👀 Awaiting Review

Development

Successfully merging this pull request may close these issues.

1 participant