Skip to content

First version of HiPS2.0 WD#4

Merged
pferniqu merged 7 commits into
ivoa-std:mainfrom
pferniqu:HIPS2.0-WD
Apr 29, 2026
Merged

First version of HiPS2.0 WD#4
pferniqu merged 7 commits into
ivoa-std:mainfrom
pferniqu:HIPS2.0-WD

Conversation

@pferniqu
Copy link
Copy Markdown
Collaborator

@pferniqu pferniqu commented Apr 1, 2026

No description provided.

@pferniqu pferniqu self-assigned this Apr 29, 2026
@andamian andamian self-requested a review April 29, 2026 17:12
andamian
andamian previously approved these changes Apr 29, 2026
Copy link
Copy Markdown
Contributor

@andamian andamian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Initial version. To be reviewed in main

@pferniqu pferniqu dismissed andamian’s stale review April 29, 2026 17:14

The merge-base changed after approval.

andamian
andamian previously approved these changes Apr 29, 2026
@pferniqu pferniqu dismissed andamian’s stale review April 29, 2026 17:36

The merge-base changed after approval.

bsipocz
bsipocz previously approved these changes Apr 29, 2026
Copy link
Copy Markdown
Member

@bsipocz bsipocz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

trying to see if I can approve, something is off with github today

@pferniqu pferniqu dismissed bsipocz’s stale review April 29, 2026 20:41

The merge-base changed after approval.

@bsipocz
Copy link
Copy Markdown
Member

bsipocz commented Apr 29, 2026

@pferniqu - is there a bot set up here or something?

@bsipocz
Copy link
Copy Markdown
Member

bsipocz commented Apr 29, 2026

Never mind, I think we had just a weird branch protection rule set up. I revised the rules and now it should be fine (but I'll also look into if this can be rebased, that may just fix that rule if as I suppose the merges may not be took into account?)

@pferniqu pferniqu merged commit 5e50101 into ivoa-std:main Apr 29, 2026
1 check passed
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.

3 participants