Skip to content

feat: outdated license link and log4j link replacement#595

Merged
zlataovce merged 7 commits into
PaperMC:mainfrom
DereC4:main
May 21, 2025
Merged

feat: outdated license link and log4j link replacement#595
zlataovce merged 7 commits into
PaperMC:mainfrom
DereC4:main

Conversation

@DereC4

@DereC4 DereC4 commented May 19, 2025

Copy link
Copy Markdown
Contributor

Hi, I saw that guava license had been renamed from COPYING 2 years ago so I fixed that, and log4j team links were leading to error pages so I dug around and pulled up the recent pages.

image

image

image

@github-actions

github-actions Bot commented May 19, 2025

Copy link
Copy Markdown
built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
papermc-docs ✅ Ready (View Log) Visit Preview cfd89ec

@olijeffers0n olijeffers0n left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is good, but the table formatting needs fixing please

@zlataovce zlataovce left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

still needs alignment fixes (see image), I recommend using a Markdown formatter to do it

image

otherwise LGTM

@DereC4

DereC4 commented May 21, 2025

Copy link
Copy Markdown
Contributor Author

will this suffice?

@zlataovce zlataovce merged commit 1216ab5 into PaperMC:main May 21, 2025
3 checks passed
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.

3 participants