Skip to content

Adds a commit hash to the "Last Updated" text at the bottom of the page#347

Merged
olijeffers0n merged 42 commits into
PaperMC:mainfrom
granny:feat/last-updated-commit
Apr 16, 2024
Merged

Adds a commit hash to the "Last Updated" text at the bottom of the page#347
olijeffers0n merged 42 commits into
PaperMC:mainfrom
granny:feat/last-updated-commit

Conversation

@granny
Copy link
Copy Markdown
Contributor

@granny granny commented Apr 8, 2024

TODO:

  • turn src/util/gitUtils.ts into a yarn patch
    • split the shell execution out into it's own method to make it easier to recreate getFileCommitHash
  • move parseFrontMatter function elsewhere?
  • better naming for LastCommitIn component and it's translation key/values or omit translation component
  • fix up namings of everything
  • fix up TODOs
  • clean up stuff left over from debugging
  • turn author into a hyperlink
  • turn hash into a hyperlink
  • replace octokit with a simple axios http request

Comment thread package.json Outdated
Comment thread src/util/authorUtils.ts Outdated
Comment thread docusaurus.config.ts Outdated
Co-authored-by: MiniDigger | Martin <admin@benndorf.dev>
@olijeffers0n olijeffers0n merged commit 1f62c35 into PaperMC:main Apr 16, 2024
@granny granny deleted the feat/last-updated-commit branch April 17, 2024 00:29
sowelipililimute pushed a commit to sowelipililimute/docs that referenced this pull request Sep 7, 2025
This PR rewrites the Review Procedure guidelines and policy. On the
whole it is largely the same, but the rewrite clarifies a lot of implied
procedure or things that weren't written down but done anyways.

Of note is the inclusion of Triaging, the `S: Conceptual Approval`
label, Maintainer Discussions, dealing with stalled discussion threads
and the inclusion of a Discord bot (as of writing not yet implemented).

This will go straight to merge, as Project Manager KeronSHB has
pre-approved this via Discord messages.
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.

4 participants