Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cleanup repo from outdated/misplaced files #1327

Merged
20 commits merged into from
Dec 13, 2023
Merged

Cleanup repo from outdated/misplaced files #1327

20 commits merged into from
Dec 13, 2023

Conversation

ghost
Copy link

@ghost ghost commented Nov 20, 2023

Here is a proposal to remove outdated files from the repository.

It'll allow us to keep things a bit more tidy and add clarity to the repo by relying more on the documentation and less on Readme.md files hidden in source code directories.

@ghost ghost added this to the 5.6.0 milestone Nov 20, 2023
@ghost ghost requested review from yflory and davidbenque November 20, 2023 15:01
@ghost
Copy link
Author

ghost commented Nov 20, 2023 •
edited by ghost
Loading

Here is a list of Readme.md files we still have laying around and require to be moved to the documentation or removed:

@ghost ghost marked this pull request as ready for review December 8, 2023 08:57
@ghost ghost changed the title Cleanup repo from outdated files Cleanup repo from outdated/misplaced files Dec 8, 2023
@wginolas
Copy link
Contributor

wginolas commented Dec 8, 2023

/www/common/onlyoffice/*/readme.md needs to stay for legal reasons

@davidbenque
Copy link
Contributor

some comments on the very different things listed

@davidbenque
Copy link
Contributor

I think that removing the files below will reduce the discoverability of information. So we could keep the readme files with a link to the place where the information has been moved

Mathilde Grünig added 7 commits December 8, 2023 11:38
@ghost ghost removed the request for review from yflory December 8, 2023 10:52
@ghost ghost self-assigned this Dec 8, 2023
@ghost ghost removed the request for review from davidbenque December 13, 2023 14:36
@ghost ghost merged commit 7026191 into staging Dec 13, 2023
@ghost ghost deleted the cleanup-repo branch December 13, 2023 14:36
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
5.6.0
Development

Successfully merging this pull request may close these issues.
2 participants
@wginolas @davidbenque