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

Use https when possible. #1597

Merged
merged 2 commits into from
Dec 27, 2020
Merged

Use https when possible. #1597

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3040ea6
Use https when possible.
XhmikosR Dec 26, 2020
fcc8f20
Readme.md: fix badges
XhmikosR Dec 26, 2020
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 CONTRIBUTING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ how we'd like to work with you:
# Documentation

Any API change should be reflected in the project's README.md file. Reuse
[jQuery's documentation](http://api.jquery.com) wherever possible, but take
[jQuery's documentation](https://api.jquery.com) wherever possible, but take
care to note aspects that make Cheerio distinct.

# Code Style
Expand Down
8 changes: 4 additions & 4 deletions Readme.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@
<h5 align="center">Fast, flexible & lean implementation of core jQuery designed specifically for the server.</h5>

<div align="center">
<a href="http://travis-ci.org/cheeriojs/cheerio">
<img src="https://secure.travis-ci.org/cheeriojs/cheerio.svg?branch=master" alt="Travis CI" />
<a href="https://travis-ci.org/cheeriojs/cheerio">
<img src="https://travis-ci.org/cheeriojs/cheerio.svg?branch=main" alt="Travis CI" />
</a>
<a href="https://coveralls.io/r/cheeriojs/cheerio">
<img src="http://img.shields.io/coveralls/cheeriojs/cheerio.svg?branch=master&style=flat" alt="Coverage" />
<img src="https://img.shields.io/coveralls/cheeriojs/cheerio.svg?branch=main&style=flat" alt="Coverage" />
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

coveralls doesn't seem to be integrated anymore. Should I drop the badge? Alternatively, we could bring coveralls back in #1600 later.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had to reset coveralls recently. I hope this will populate again soon.

</a>
<a href="https://gitter.im/cheeriojs/cheerio?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge">
<img src="https://badges.gitter.im/Join%20Chat.svg" alt="Join the chat at https://gitter.im/cheeriojs/cheerio" />
Expand Down Expand Up @@ -259,7 +259,7 @@ Cheerio collections are made up of objects that bear some resemblance to [browse

## Screencasts

http://vimeo.com/31950192
https://vimeo.com/31950192

> This video tutorial is a follow-up to Nettut's "How to Scrape Web Pages with Node.js and jQuery", using cheerio instead of JSDOM + jQuery. This video shows how easy it is to use cheerio and how much faster cheerio is than JSDOM + jQuery.

Expand Down
20 changes: 10 additions & 10 deletions lib/api/attributes.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ function setAttr(el, name, value) {
*
* @param {string} name - Name of the attribute.
* @param {string} [value] - If specified sets the value of the attribute.
* @see {@link http://api.jquery.com/attr/}
* @see {@link https://api.jquery.com/attr/}
*
* @returns {string | Cheerio} If `value` is specified the instance itself,
* otherwise the attribute's value.
Expand Down Expand Up @@ -176,7 +176,7 @@ function setProp(el, name, value) {
*
* @param {string} name - Name of the property.
* @param {any} [value] - If specified set the property to this.
* @see {@link http://api.jquery.com/prop/}
* @see {@link https://api.jquery.com/prop/}
*
* @returns {string | Cheerio} If `value` is specified the instance itself,
* otherwise the prop's value.
Expand Down Expand Up @@ -323,7 +323,7 @@ function readData(el, name) {
*
* @param {string} name - Name of the attribute.
* @param {any} [value] - If specified new value.
* @see {@link http://api.jquery.com/data/}
* @see {@link https://api.jquery.com/data/}
*
* @returns {string | Cheerio} If `value` is specified the instance itself,
* otherwise the data attribute's value.
Expand Down Expand Up @@ -367,7 +367,7 @@ exports.data = function (name, value) {
* //=> <input type="text" value="test"/>
*
* @param {string} [value] - If specified new value.
* @see {@link http://api.jquery.com/val/}
* @see {@link https://api.jquery.com/val/}
*
* @returns {string | Cheerio} If a new `value` is specified the instance
* itself, otherwise the value.
Expand Down Expand Up @@ -445,7 +445,7 @@ function splitNames(names) {
* //=> <li>Apple</li>
*
* @param {string} name - Name of the attribute.
* @see {@link http://api.jquery.com/removeAttr/}
* @see {@link https://api.jquery.com/removeAttr/}
*
* @returns {Cheerio} The instance itself.
*/
Expand Down Expand Up @@ -475,7 +475,7 @@ exports.removeAttr = function (name) {
* //=> true
*
* @param {string} className - Name of the class.
* @see {@link http://api.jquery.com/hasClass/}
* @see {@link https://api.jquery.com/hasClass/}
*
* @returns {boolean} Indicates if an element has the given `className`.
*/
Expand Down Expand Up @@ -510,7 +510,7 @@ exports.hasClass = function (className) {
* //=> <li class="apple fruit red">Apple</li>
*
* @param {string} value - Name of new class.
* @see {@link http://api.jquery.com/addClass/}
* @see {@link https://api.jquery.com/addClass/}
*
* @returns {Cheerio} The instance itself.
*/
Expand Down Expand Up @@ -567,7 +567,7 @@ exports.addClass = function (value) {
* //=> <li class="">Apple</li>
*
* @param {string} value - Name of the class.
* @see {@link http://api.jquery.com/removeClass/}
* @see {@link https://api.jquery.com/removeClass/}
*
* @returns {Cheerio} The instance itself.
*/
Expand Down Expand Up @@ -629,7 +629,7 @@ exports.removeClass = function (value) {
*
* @param {string | Function} value - Name of the class. Can also be a function.
* @param {boolean} [stateVal] - If specified the state of the class.
* @see {@link http://api.jquery.com/toggleClass/}
* @see {@link https://api.jquery.com/toggleClass/}
*
* @returns {Cheerio} The instance itself.
*/
Expand Down Expand Up @@ -687,7 +687,7 @@ exports.toggleClass = function (value, stateVal) {
* so `this` refers to the current element.
*
* @param {string | Function | Cheerio | Node} selector - Selector for the selection.
* @see {@link http://api.jquery.com/is/}
* @see {@link https://api.jquery.com/is/}
*
* @returns {Cheerio} The instance itself.
*/
Expand Down
2 changes: 1 addition & 1 deletion lib/api/css.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ var toString = Object.prototype.toString;
*
* @param {string | object} prop - The name of the property.
* @param {string} [val] - If specified the new value.
* @see {@link http://api.jquery.com/css/}
* @see {@link https://api.jquery.com/css/}
*
* @returns {Cheerio} The instance itself.
*/
Expand Down
4 changes: 2 additions & 2 deletions lib/api/forms.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ var rCRLF = /\r?\n/g;
/**
* Encode a set of form elements as a string for submission.
*
* @see {@link http://api.jquery.com/serialize/}
* @see {@link https://api.jquery.com/serialize/}
*
* @returns {string} The serialized form.
*/
Expand All @@ -33,7 +33,7 @@ exports.serialize = function () {
* $('<form><input name="foo" value="bar" /></form>').serializeArray();
* //=> [ { name: 'foo', value: 'bar' } ]
*
* @see {@link http://api.jquery.com/serializeArray/}
* @see {@link https://api.jquery.com/serializeArray/}
*
* @returns {object[]} The serialized form.
*/
Expand Down
32 changes: 16 additions & 16 deletions lib/api/manipulation.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ function uniqueSplice(array, spliceIdx, spliceCount, newElems, parent) {
* // </ul>
*
* @param {string | Cheerio} target - Element to append elements to.
* @see {@link http://api.jquery.com/appendTo/}
* @see {@link https://api.jquery.com/appendTo/}
*
* @returns {Cheerio} The instance itself.
*/
Expand Down Expand Up @@ -162,7 +162,7 @@ exports.appendTo = function (target) {
* // </ul>
*
* @param {string | Cheerio} target - Element to prepend elements to.
* @see {@link http://api.jquery.com/prependTo/}
* @see {@link https://api.jquery.com/prependTo/}
*
* @returns {Cheerio} The instance itself.
*/
Expand Down Expand Up @@ -196,7 +196,7 @@ exports.prependTo = function (target) {
*
* @function
*
* @see {@link http://api.jquery.com/append/}
* @see {@link https://api.jquery.com/append/}
*/
exports.append = _insert(function (dom, children, parent) {
uniqueSplice(children, children.length, 0, dom, parent);
Expand All @@ -217,7 +217,7 @@ exports.append = _insert(function (dom, children, parent) {
*
* @function
*
* @see {@link http://api.jquery.com/prepend/}
* @see {@link https://api.jquery.com/prepend/}
*/
exports.prepend = _insert(function (dom, children, parent) {
uniqueSplice(children, 0, 0, dom, parent);
Expand Down Expand Up @@ -303,7 +303,7 @@ function _wrap(insert) {
*
* @param {Cheerio} wrapper - The DOM structure to wrap around each element in
* the selection.
* @see {@link http://api.jquery.com/wrap/}
* @see {@link https://api.jquery.com/wrap/}
*/
exports.wrap = _wrap(function (el, elInsertLocation, wrapperDom) {
var parent = el.parent;
Expand Down Expand Up @@ -353,7 +353,7 @@ exports.wrap = _wrap(function (el, elInsertLocation, wrapperDom) {
*
* @param {Cheerio} wrapper - The DOM structure to wrap around the content of
* each element in the selection.
* @see {@link http://api.jquery.com/wrapInner/}
* @see {@link https://api.jquery.com/wrapInner/}
*/
exports.wrapInner = _wrap(function (el, elInsertLocation, wrapperDom) {
updateDOM(el.children, elInsertLocation);
Expand Down Expand Up @@ -453,7 +453,7 @@ exports.wrapAll = function (wrapper) {
* // <li class="pear">Pear</li>
* // </ul>
*
* @see {@link http://api.jquery.com/after/}
* @see {@link https://api.jquery.com/after/}
*
* @returns {Cheerio} The instance itself.
*/
Expand Down Expand Up @@ -499,7 +499,7 @@ exports.after = function () {
* // </ul>
*
* @param {string | Cheerio} target - Element to insert elements after.
* @see {@link http://api.jquery.com/insertAfter/}
* @see {@link https://api.jquery.com/insertAfter/}
*
* @returns {Cheerio} The set of newly inserted elements.
*/
Expand Down Expand Up @@ -549,7 +549,7 @@ exports.insertAfter = function (target) {
* // <li class="pear">Pear</li>
* // </ul>
*
* @see {@link http://api.jquery.com/before/}
* @see {@link https://api.jquery.com/before/}
*
* @returns {Cheerio} The instance itself.
*/
Expand Down Expand Up @@ -595,7 +595,7 @@ exports.before = function () {
* // </ul>
*
* @param {string | Cheerio} target - Element to insert elements before.
* @see {@link http://api.jquery.com/insertBefore/}
* @see {@link https://api.jquery.com/insertBefore/}
*
* @returns {Cheerio} The set of newly inserted elements.
*/
Expand Down Expand Up @@ -645,7 +645,7 @@ exports.insertBefore = function (target) {
* // </ul>
*
* @param {string} [selector] - Optional selector for elements to remove.
* @see {@link http://api.jquery.com/remove/}
* @see {@link https://api.jquery.com/remove/}
*
* @returns {Cheerio} The instance itself.
*/
Expand Down Expand Up @@ -675,7 +675,7 @@ exports.remove = function (selector) {
* // </ul>
*
* @param {Cheerio | Function} content - Replacement for matched elements.
* @see {@link http://api.jquery.com/replaceWith/}
* @see {@link https://api.jquery.com/replaceWith/}
*
* @returns {Cheerio} The instance itself.
*/
Expand Down Expand Up @@ -714,7 +714,7 @@ exports.replaceWith = function (content) {
* $.html();
* //=> <ul id="fruits"></ul>
*
* @see {@link http://api.jquery.com/empty/}
* @see {@link https://api.jquery.com/empty/}
*
* @returns {Cheerio} The instance itself.
*/
Expand All @@ -740,7 +740,7 @@ exports.empty = function () {
* //=> <li class="mango">Mango</li>
*
* @param {string} str - If specified used to replace selection's contents.
* @see {@link http://api.jquery.com/html/}
* @see {@link https://api.jquery.com/html/}
*
* @returns {Cheerio} The instance itself.
*/
Expand Down Expand Up @@ -789,7 +789,7 @@ exports.toString = function () {
* // Pear
*
* @param {string} [str] - If specified replacement for the selected element's contents.
* @see {@link http://api.jquery.com/text/}
* @see {@link https://api.jquery.com/text/}
*
* @returns {Cheerio | string} The instance itself when setting text, otherwise
* the rendered document.
Expand Down Expand Up @@ -823,7 +823,7 @@ exports.text = function (str) {
* @example
* const moreFruit = $('#fruits').clone();
*
* @see {@link http://api.jquery.com/clone/}
* @see {@link https://api.jquery.com/clone/}
*
* @returns {Cheerio} The cloned object.
*/
Expand Down
Loading