Skip to content

Edit of the repositories overview page #1488

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.

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

Closed
wants to merge 21 commits into from

Conversation

flexseth
Copy link
Collaborator

@flexseth flexseth commented Jun 6, 2024

Motivation for the change, related issues

update of copy to reflect what was found on the repositories with a good voice. A collaboration between @ironnysh, @adamziel and myself.

Implementation details

discuss changes needed and approve PR

Testing Instructions (or ideally a Blueprint)

minor formatting changes needed, feedback needed

flexseth and others added 17 commits April 14, 2024 19:38
Less wordy and more direct
but don't go into too much detail
I tried to balance the objective of the original issue (WordPress#1259) and this PR:
- @adamziel's intention (as I understand it) is to clarify the ecosystem:
> Let's write a page that explains the structure of the larger Playground initiative, especially around GitHub repositories and what are they for.

- @flexseth's PR focuses on "action":
> guide users to submitting issues as accurately as possible 

To achieve this, I did the following:
1. Reorder the repos by relevance.
2. Add an intro.
3. Revise the content to describe "what you'll find here" _and_ how to participate (contribute/get help).

Finally, I removed redundant references, fixed formatting, and polished the wording.
@ironnysh
Copy link
Collaborator

ironnysh commented Jun 6, 2024

@flexseth, there are two files... thought your plan was to merge both

@flexseth
Copy link
Collaborator Author

flexseth commented Jun 6, 2024

@flexseth, there are two files... thought your plan was to merge both

Good catch, removed the duplicate file. I used some of the text you provided, some of what I came up with, and some of what came from the repos (but simplified due to links for more info)

Let me know what you think!

I'll request review from @adamziel on Friday, so please review ASAP if you can. Trying to get this in before WCEU if possible

@flexseth flexseth requested a review from adamziel June 7, 2024 18:12
@flexseth
Copy link
Collaborator Author

flexseth commented Jul 3, 2024

I'll try to take another look at this over the weekend and submit for review, thanks for the ping, @bgrgicak

@adamziel
Copy link
Collaborator

adamziel commented Jul 12, 2024

Looping in @juanmaguitar who's overhauling the documentation for reviews here

@adamziel
Copy link
Collaborator

As of #1602, the ecosystem information is now contextually included in the relevant documentation sections which effectively solves the same problem. I'll go ahead and close this PR.

@adamziel adamziel closed this Aug 27, 2024
@flexseth
Copy link
Collaborator Author

flexseth commented Sep 7, 2024

Thanks everyone for collaborating on this. I started school earlier this year and did poorly my first semester, made Dean's List last semester and on track again for that. Looking to get involved for contributor day if time permits!

Cheers/Seth

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