Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<!-- Thanks for contributing! Before you submit your pull request, please make sure to check the following boxes by putting an x in the [ ] (don't: [x ], [ x], do: [x]); you should remove sections for files you aren't changing -->
<!-- Thanks for contributing! Before you submit your pull request, please make sure to check the following boxes by putting an x in the [ ] (do: `[x]`, don't: `[x ]` or `[ x]`); you should remove sections for files you aren't changing -->

### Overall Checklist
- [ ] I agree to the project's [Developer Certificate of Origin](https://github.com/apple/password-manager-resources/blob/main/DEVELOPER_CERTIFICATE_OF_ORIGIN.md)
Expand Down
18 changes: 18 additions & 0 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,24 @@ When adding a data item to a top-level JSON object, please keep keys alphabetize

One more point to consider is to keep your pull requests small and limited in scope to a specific change. (For example, it's best to submit a "Password Rule" for a single website or group of related websites in one pull request and not conflate unrelated changes that can prevent other from being merged.)

### Filling Out the Pull Request Template

When you open a pull request, its description starts out with the [pull request template](.github/PULL_REQUEST_TEMPLATE.md). The template has an "Overall Checklist" section and a "for <file name>" section for each file in [`quirks/`](quirks/). Each section lists statements that you agree to by checking their checkboxes (`[ ]` -> `[x]`).

Your pull request cannot be merged until the pull request template is properly filled out. This is enforced by a "Pull Request Template" check that runs against every pull request.

Specifically:

* Keep the "Overall Checklist" section, and the "for <file name>" section for each file in [`quirks/`](quirks/) that your pull request adds to. You should remove the sections for files you aren't changing.
* For each checkbox, reason about the included statement and ensure that it is true by reviewing your change or gathering additional documentation. Once you're sure that a statement is true, check the statement's checkbox by putting an `x` between its brackets (do: `[x]`, don't: `[x ]` or `[ x]`).
* Where checkboxes are grouped under "One or more of the following:", check at least one that applies.
* Fill in each blank field, like "Web browser name:", on the same line as its label.
* Do not reword or remove the checkboxes in the sections you keep.

If the check fails, its output explains what's missing. Edit the pull request's description to fix it, and the check will run again.

These requirements apply to every pull request, including ones prepared or opened by AI agents and other tools. If you use one of these tools, make sure it starts the description from the template and preserves the template's sections, checkboxes, and fields, rather than replacing them with its own summary, and make sure that each checked box is accurate. Humans are accountable for the changes made in this project, and by checking the checkboxes in the pull request template, a human is asserting the accuracy of each statement. If a contributor checks a box for a statement that isn't true, or tries to bypass or abuse this check, they may be banned from the project.

### Contributing Password Rules

Contributing a password rule is appropriate if a service doesn't accept a password generated by your password manager with its default settings or with a reasonable configuration. Contributing a rule involves writing a rule, testing it, documenting your investigation, and submitting a pull request.
Expand Down
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,8 @@ A password manager may wish to not offer to save a new password submitted in suc

Please review [how to contribute](CONTRIBUTING.md) if you would like to submit a pull request.

Pull request descriptions must preserve the [pull request template](.github/PULL_REQUEST_TEMPLATE.md) and have its checkboxes checked off. This applies to pull requests prepared by AI agents and other tools, too. See [Filling Out the Pull Request Template](CONTRIBUTING.md#filling-out-the-pull-request-template) for details.

## Asking Questions and Discussing Ideas

If you have any questions you'd like to ask publicly, or ideas you'd like to discuss, please [raise a GitHub issue](https://github.com/apple/password-manager-resources/issues) or send a message in the project's [Slack instance](https://pw-manager-resources.slack.com). Anyone participating in the project is welcome to join the Slack instance by [emailing the project's maintainers at Apple](mailto:password-manager-resources-maintainers@apple.com) and asking for an invitation. Please include your GitHub user name when you do this.
Expand Down
Loading