Skip to content

Add alternative alignment for CDS - #178

Merged
fxbuson merged 2 commits into
masterfrom
cds-altalign
Oct 18, 2026
Merged

fxbuson merged 2 commits into
masterfrom
cds-altalign

Conversation

@fxbuson

@fxbuson fxbuson commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

As discussed on #177 we'd like to add an explicit accepted alternate alignment for the CDS glyph (and thus compliant with best practices).

Feel free to propose different wording since I know the controlled vocabulary is quite important for the spec

@fxbuson
fxbuson requested review from cjmyers and jakebeal October 15, 2026 13:41
@fxbuson fxbuson self-assigned this Oct 15, 2026
@cjmyers

cjmyers commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

While I approved this, the builds are failing. Looks like something needs to be updated. Please sort this before merging this PR.

@fxbuson

fxbuson commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

@cjmyers Should I add changes to address the Github actions on this PR or make a new one?

@cjmyers

cjmyers commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

I think it is fine to fix this PR

@cjmyers

cjmyers commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

Passing now. Should be safe to merge.

@fxbuson
fxbuson merged commit f6b2f53 into master Oct 18, 2026
3 checks 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