-
Notifications
You must be signed in to change notification settings - Fork 320
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
Refresh Contributor Day Page for November 2023 Hack Day #470
Conversation
contributor-day.md
Outdated
``` | ||
git clone https://github.com/wp-cli/wp-cli-dev | ||
|
||
Before you can proceed further, you'll need to make sure you have [Composer](https://getcomposer.org/), PHP, and a functioning MySQL or MariaDB server on your local machine too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we provide more direction or links here?
Also, it would be nice to have a bin/check-environment
script in wp-cli-dev
that validates all of the environmental constraints are met.
contributor-day.md
Outdated
```bash | ||
cd core-command | ||
gh repo fork | ||
``` | ||
|
||
All WP-CLI pull requests are expected to have tests. See [running and writing tests](https://make.wordpress.org/cli/handbook/contributions/pull-requests/#running-and-writing-tests) for a complete introduction. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It'd be nice to have a Behat explanation linked here.
|
||
**Seasoned contributors** | ||
|
||
- [Run the test suite with SQLite](https://github.com/wp-cli/wp-cli/issues/5831) | ||
- [Set WP_CLI_PACKAGES_DIR in config file](https://github.com/wp-cli/wp-cli/issues/5645) | ||
- [Improve speed of import when uploads are available locally](https://github.com/wp-cli/import-command/issues/83) | ||
- [Introduce a dedicated search-replace url command](https://github.com/wp-cli/search-replace-command/issues/186) | ||
- [Path errors when running wp dist-archive on Windows](https://github.com/wp-cli/dist-archive-command/issues/76) | ||
- [Prompt if archive file already exists, instead of overwriting automatically](https://github.com/wp-cli/dist-archive-command/issues/70) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are there other issues we should add to this list for seasoned contributors?
contributor-day.md
Outdated
Your table leads for the day are: [schlessera](https://github.com/schlessera), [danielbachhuber](https://github.com/danielbachhuber) | ||
We put together this guide to make contributing as straightforward as possible. Please also join the [#cli channel in WordPress.org Slack](https://wordpress.slack.com/messages/C02RP4T41) ([sign up instructions](https://make.wordpress.org/chat/)) to chat with other contributors, for questions, etc. | ||
|
||
Your leads for the day are: [schlessera](https://github.com/schlessera), [danielbachhuber](https://github.com/danielbachhuber), [swissspidy] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we link the last nickname to the GH profile too?
Co-authored-by: Pascal Birchler <pascalb@google.com>
See wp-cli/wp-cli#5856