Skip to content

Conversation

@zecakeh
Copy link
Contributor

@zecakeh zecakeh commented Feb 21, 2025

I am not sure if it closes #1612. It says "somewhere on the page", so on every page of the spec, but here we do it only on the unstable changelog page, which might be enough?

If we want to show it on every page, it will probably require to copy the commit hash in the config file or something like that.

Pull Request Checklist

Preview: https://pr2078--matrix-spec-previews.netlify.app

@zecakeh zecakeh requested a review from a team as a code owner February 21, 2025 16:36
@zecakeh zecakeh marked this pull request as draft February 21, 2025 16:42
@zecakeh
Copy link
Contributor Author

zecakeh commented Feb 21, 2025

Actually this doesn't work because the unstable changelog file is not committed to the repository.

@zecakeh zecakeh marked this pull request as ready for review February 21, 2025 17:15
@zecakeh
Copy link
Contributor Author

zecakeh commented Feb 21, 2025

It's fixed now.

Signed-off-by: Kévin Commaille <[email protected]>
@zecakeh zecakeh requested a review from richvdh March 4, 2025 19:24
@richvdh richvdh merged commit 4ff9a85 into matrix-org:main Mar 5, 2025
12 checks passed
@zecakeh zecakeh deleted the unstable-commit branch March 5, 2025 10:01
@Johennes Johennes mentioned this pull request Jul 2, 2025
30 tasks
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.

Unstable spec should include the git commit hash somewhere on the page

2 participants