Update documentation and issue- and PR- templates.
This commit is contained in:
parent
4de550c882
commit
0e909f2948
83
.github/CONTRIBUTING.md
vendored
83
.github/CONTRIBUTING.md
vendored
@ -0,0 +1,83 @@
|
|||||||
|
# We've moved!
|
||||||
|
|
||||||
|
You can now find the latest code, issues, and releases at [RipMeApp/ripme](https://github.com/RipMeApp/ripme/).
|
||||||
|
|
||||||
|
|
||||||
|
# Etiquette
|
||||||
|
|
||||||
|
Please be polite and supportive to all users and contributors. Please be inclusive of everyone regardless of race, religion, gender identity or expression, sexual preference, or tools and platform preferences. Please be helpful and stick to the engineering facts, and avoid expressing unhelpful or off-topic opinions.
|
||||||
|
|
||||||
|
Many of the sites we deal with contain NSFW (Not Safe For Work) content. Please assume any link you see is NSFW unless tagged otherwise -- i.e., SFW (Safe For Work). Please tag all links you post with either (NSFW) or (SFW) to be considerate to others who may not be browsing this repo in private.
|
||||||
|
|
||||||
|
|
||||||
|
# Priorities
|
||||||
|
|
||||||
|
Our priorities, roughly in order of impact:
|
||||||
|
|
||||||
|
* Bug fixes for popular websites (e.g. that have recently changed their layout and broke our behavior).
|
||||||
|
* Bug fixes for minor websites
|
||||||
|
* New Rippers
|
||||||
|
* Refactorings that make development easier
|
||||||
|
* Style fixes
|
||||||
|
|
||||||
|
|
||||||
|
# Issues
|
||||||
|
|
||||||
|
## Bugs
|
||||||
|
|
||||||
|
If you have noticed a bug in RipMe, please open an issue at [RipMeApp/ripme](https://github.com/RipMeApp/ripme/issues/new).
|
||||||
|
|
||||||
|
Please include enough information that we can easily confirm the problem and verify when it is fixed. (For example: the exact URL that you tried to rip when something bad/incorrect happened.)
|
||||||
|
|
||||||
|
|
||||||
|
## Rippers / Website Support
|
||||||
|
|
||||||
|
Request support for more sites by adding a comment to [this Github issue](https://github.com/4pr0n/ripme/issues/502).
|
||||||
|
|
||||||
|
If you're a developer, you can add your own Ripper by following the wiki guide
|
||||||
|
[How To Create A Ripper for HTML Websites](https://github.com/ripmeapp/ripme/wiki/How-To-Create-A-Ripper-for-HTML-websites).
|
||||||
|
|
||||||
|
|
||||||
|
# Pull Requests
|
||||||
|
|
||||||
|
Before you open your pull request, please consider the following:
|
||||||
|
|
||||||
|
* Please ensure your change is based on the `master` branch of this repo (i.e. https://github.com/RipMeApp/ripme.git)
|
||||||
|
* Please do `git pull` on the `master` branch before starting work on your bug fix.
|
||||||
|
* This helps avoid merge conflicts.
|
||||||
|
* Please ensure your change includes only the minimum changes needed to fix a single issue. These are easiest to review and tend to get merged more quickly. If the scope of your PR is wider than a single issue, you may be asked to reopen your PR as multiple separate PRs.
|
||||||
|
* Are you fixing an issue from one of the issue trackers ([RipMeApp](https://github.com/RipMeApp/ripme/issues) or [4pr0n](https://github.com/4pr0n/ripme/issues))? If so, please ensure that you reference the issue you are fixing in your commit message so that it will be [automatically closed](https://help.github.com/articles/closing-issues-via-commit-messages/).
|
||||||
|
* Please ensure you verifying that you did not break any functionality outside of your change or feature.
|
||||||
|
* The CI might be broken, so please ensure that `mvn test` shows no new errors since before your change.
|
||||||
|
* Keep in mind each Ripper likely support multiple URL formats for each website, which have different content layouts (users, galleries, etc.)
|
||||||
|
* We deal with live websites, so things might break while we aren't looking. Websites can change and content can be deleted at any time. Our code and/or tests may need to be rewritten to fix issues.
|
||||||
|
* Please consider adding a test to check for regressions to the Ripper you added or the bug you fixed.
|
||||||
|
* See e.g. src/test/java/com/rarchives/ripme/tst/ripper/rippers/ImgurRipperTest.java
|
||||||
|
|
||||||
|
|
||||||
|
## Style
|
||||||
|
|
||||||
|
Generally, we will regard style changes as low-priority. Please consider that the contributors don't have a lot of volunteer time to work on this project, so style changes which do not improve the functionality of the project may be ignored in favor of critical bug fixes, new features, or other tangible improvements. Additionally, changes which are difficult to review may be ignored.
|
||||||
|
|
||||||
|
If you make a large or complex change, please detail what changes you made, and how they are helpful or what they fix.
|
||||||
|
|
||||||
|
If you feel the need to make a style change: changes to spacing and so on are easy to review regardless of the number of lines changed if ONLY whitespace changes are present. (`git diff -w` is helpful here.) If you change spacing and layout, please avoid also moving things around or otherwise refactoring the code. **Submit refactoring changes separately from style changes.**
|
||||||
|
|
||||||
|
Good style is a tool for communicating your intent with other developers of the project. We are interested in maintaining reasonably well-styled code. If a contribution is illegible, we may refuse to merge it until it has been brought up to reasonable style guidelines. If a contribution violates any of our "rules" but is still legible, it is likely to be merged anyway.
|
||||||
|
|
||||||
|
Some recommendations:
|
||||||
|
|
||||||
|
* Above all, be consistent!
|
||||||
|
* Spaces, not tabs.
|
||||||
|
* We prefer "Egyptian brackets":
|
||||||
|
* `if (...) {`
|
||||||
|
* `} else {`
|
||||||
|
* `} else if (...) {`
|
||||||
|
* `}`
|
||||||
|
* Constants in `UPPER_SNAKE_CASE`
|
||||||
|
* Class names in `PascalCase` a.k.a. `UpperCamelCase`
|
||||||
|
* Variable names in `camelCase` a.k.a. `lowerCamelCase`
|
||||||
|
* Do not use Hungarian notation
|
||||||
|
* Do not use `lower_snake_case`
|
||||||
|
* Place spaces around binary operators: `1 + 2` not `1+2`
|
||||||
|
* Do not place spaces inside of parens: `(a + b)` not `( a + b )`
|
3
.github/ISSUE_TEMPLATE.md
vendored
3
.github/ISSUE_TEMPLATE.md
vendored
@ -1,4 +1,7 @@
|
|||||||
<!--
|
<!--
|
||||||
|
We've moved! If you are not already, please consider opening your issue at the following link:
|
||||||
|
https://github.com/RipMeApp/ripme/issues/new
|
||||||
|
|
||||||
If this is a bug, please fill out the information below.
|
If this is a bug, please fill out the information below.
|
||||||
Please include any additional information that would help us fix the bug.
|
Please include any additional information that would help us fix the bug.
|
||||||
If this is a feature request or other type of issue, provide whatever information you feel is appropriate.
|
If this is a feature request or other type of issue, provide whatever information you feel is appropriate.
|
||||||
|
33
.github/PULL_REQUEST_TEMPLATE.md
vendored
33
.github/PULL_REQUEST_TEMPLATE.md
vendored
@ -0,0 +1,33 @@
|
|||||||
|
<!--
|
||||||
|
We've moved! If you are not already, please consider opening your pull request here:
|
||||||
|
https://github.com/RipMeApp/ripme/
|
||||||
|
|
||||||
|
To help us verify your change, please fill out the information below.
|
||||||
|
-->
|
||||||
|
|
||||||
|
# Category
|
||||||
|
|
||||||
|
This change is exactly one of the following (please change `[ ]` to `[x]`) to indicate which:
|
||||||
|
* [ ] a bug fix -- which issue? #...
|
||||||
|
* [ ] a new Ripper
|
||||||
|
* [ ] a refactoring
|
||||||
|
* [ ] a style change/fix
|
||||||
|
|
||||||
|
|
||||||
|
# Description
|
||||||
|
|
||||||
|
Please add details about your change here.
|
||||||
|
|
||||||
|
|
||||||
|
# Testing
|
||||||
|
|
||||||
|
Required verification:
|
||||||
|
* [ ] I've verified that there are no regressions in `mvn test` (there are no new failures or errors).
|
||||||
|
* [ ] I've verified that this change works as intended.
|
||||||
|
* [ ] Downloads all relevant content.
|
||||||
|
* [ ] Downloads content from multiple pages (as necessary or appropriate).
|
||||||
|
* [ ] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.
|
||||||
|
* [ ] I've verified that this change did not break existing functionality (especially in the Ripper I modified).
|
||||||
|
|
||||||
|
Optional but recommended:
|
||||||
|
* [ ] I've added a unit test to cover my change.
|
@ -48,7 +48,7 @@ For information about running the `.jar` file, see [the How To Run wiki](https:/
|
|||||||
|
|
||||||
Request support for more sites by adding a comment to [this Github issue](https://github.com/4pr0n/ripme/issues/502).
|
Request support for more sites by adding a comment to [this Github issue](https://github.com/4pr0n/ripme/issues/502).
|
||||||
|
|
||||||
If you're a developer, you can add your own by following the wiki guide
|
If you're a developer, you can add your own Ripper by following the wiki guide
|
||||||
[How To Create A Ripper for HTML Websites](https://github.com/ripmeapp/ripme/wiki/How-To-Create-A-Ripper-for-HTML-websites).
|
[How To Create A Ripper for HTML Websites](https://github.com/ripmeapp/ripme/wiki/How-To-Create-A-Ripper-for-HTML-websites).
|
||||||
|
|
||||||
# Compiling & Building
|
# Compiling & Building
|
||||||
|
Loading…
Reference in New Issue
Block a user