Skip to content

readme: add CI badges - #72

Open
illwieckz wants to merge 2 commits into
masterfrom
illwieckz/readme
Open

illwieckz wants to merge 2 commits into
masterfrom
illwieckz/readme

Conversation

@illwieckz

Copy link
Copy Markdown
Member

Add CI badges now that we have working CI.

@illwieckz

Copy link
Copy Markdown
Member Author

The badge are not seen from GitHub but seen when I'm logged in into Azure. I noticed the Azure projet is set to “private” instead of “public”, I don't know how to change that. That may also explain why the CI is so slow because it does only one CI job at a time unlike our public Azure projects.

@illwieckz

Copy link
Copy Markdown
Member Author

OK, I could enable the anonymous badges at organization level to be able to enable them at the project level.

It would be better if the project was public.

@slipher

slipher commented Oct 4, 2026 •

Copy link
Copy Markdown
Member

I guess we don't want these in view of Unvanquished/Unvanquished#3577.

@illwieckz
illwieckz changed the base branch from master to slipher/ci-convert October 5, 2026 00:01
@illwieckz

Copy link
Copy Markdown
Member Author

I now ported the CI badges to the ported GitHub Actions CI.

Also fixed some formatting, the platform status wasn't printed one platform per line, I could have add line breaking two-spaces ending but went the list way. I also unified the whitespacing (two empty lines before a title, like the convention used in our other READMEs and already partially implemented there).

@illwieckz

Copy link
Copy Markdown
Member Author

To be fully tested one should wait for #73 to be merged to make sure badges are available and generated:

Base automatically changed from slipher/ci-convert to master October 5, 2026 04:39
@illwieckz
illwieckz force-pushed the illwieckz/readme branch 2 times, most recently from 2de8d45 to 7cdecd5 Compare October 6, 2026 00:16
@illwieckz

Copy link
Copy Markdown
Member Author

So there was a small mistake in the badge URLs, they're now working.

@illwieckz

Copy link
Copy Markdown
Member Author

As a side note I notice it runs the CI even when the code isn't modified.

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.

2 participants