Skip to content

Conversation

@xamyl
Copy link
Contributor

@xamyl xamyl commented Apr 5, 2025

The site content is ... A Next.js homepage for my ThemaKit project.

@xamyl
Copy link
Contributor Author

xamyl commented Apr 5, 2025

I've tried 4 times and it still doesn't validate for whatever reason.

@MattIPv4
Copy link
Member

MattIPv4 commented Apr 5, 2025

Are you actually reading the CI error message? Your entry just needs to be in alphabetical order...

@xamyl
Copy link
Contributor Author

xamyl commented Apr 5, 2025

There we go! I do not know how to read the error message, that is why. Sorry!

Co-authored-by: Matt Cowley <me@mattcowley.co.uk>
@xamyl
Copy link
Contributor Author

xamyl commented Apr 5, 2025

did not realize that, sorry for the trouble

Copy link
Member

@MattIPv4 MattIPv4 left a comment

Choose a reason for hiding this comment

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

✅ Commit valid, CNAME present, site appears to have limited content *

* based off https://github.com/themakit/themakit.github.io/blob/main/src/pages/index.tsx

@MattIPv4 MattIPv4 added add This PR/issue is regarding adding a new subdomain organisation This PR/issue is regarding a target on a GitHub organisation labels Apr 5, 2025
@indus
Copy link
Member

indus commented Apr 5, 2025

"There we go!" 😁 - Welcome https://thema.js.org

@indus indus merged commit 173ddf2 into js-org:master Apr 6, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add This PR/issue is regarding adding a new subdomain organisation This PR/issue is regarding a target on a GitHub organisation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants