Skip to content

readme: redo the CI badges and reword things a bit - #2007

Open
illwieckz wants to merge 1 commit into
masterfrom
illwieck/readme
Open

illwieckz wants to merge 1 commit into
masterfrom
illwieck/readme

Conversation

@illwieckz

@illwieckz illwieckz commented Oct 11, 2026 •

Copy link
Copy Markdown
Member

Redo the CI badges after:

The GitHub Action badges should now be per job. I'm not sure we can get a direct link to the job though.
The Appveyor badges are still per-project, they're still valuable because once it's green, each of them is green. I used a diferent URL for the image that references the GitHub instance and not the Appveyor profile, so the image path should survive even if we move Appveyor to another profile (the history link will not).

I believe it's good enough for now. It's already better than before.

Edit: I also reworked and unified some wordings with the game code side:

@slipher

slipher commented Oct 11, 2026

Copy link
Copy Markdown
Member

The 32-bit Windows job was nuked at some point so there is only one Appveyor job anyway.

@illwieckz

Copy link
Copy Markdown
Member Author

Ah yes, thanks.

@illwieckz

illwieckz commented Oct 12, 2026 •

Copy link
Copy Markdown
Member Author

I removed the mistaken windows-i686-msvc badge. Since there was an empty cell in the table I put there a badge for the CodeQL C++ static analysis because why not.

@illwieckz

Copy link
Copy Markdown
Member Author

I noticed the Web link was mistakenly linking to the forum instead of the blog, also fixed.

@illwieckz
illwieckz force-pushed the illwieck/readme branch 2 times, most recently from a1e52ee to 28fce28 Compare October 12, 2026 03:10
@illwieckz illwieckz changed the title readme: redo the CI badges readme: redo the CI badges and reword things a bit Oct 12, 2026
@illwieckz

Copy link
Copy Markdown
Member Author

I also reworked and unified some wordings with the game code side:

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