Summary

Fix line length issues in our JavaScript code.

Review Request #12767 — Created Jan. 6, 2023 and submitted Jan. 8, 2023, 10:17 p.m.

Information

Owner

chipx86

Repository

Djblets

Branch

release-4.x

Bugs

Depends On

Reviewers

Groups

djblets

People

Description

We had a number of places where the line length was slightly too long.
This updates various doc comments and code to fit within the max line
length, satisfying ESLint.

Some lines are still above the limit, such as legacy gettext() calls,
which cannot wrap.

Testing Done

Verified ESLint no longer found (relevant) line length issues.

Commits

Summary	ID
Fix line length issues in our JavaScript code. We had a number of places where the line length was slightly too long. This updates various doc comments and code to fit within the max line length, satisfying ESLint.	a23e9270205e93a76642fa7092b1157e0bad8589

flake8 passed.

JSHint passed.

Change Summary:

Fixed up some additional lines (which I missed during git add -p).

Commits:

	Summary	ID
	Fix line length issues in our JavaScript code. We had a number of places where the line length was slightly too long. This updates various doc comments and code to fit within the max line length, satisfying ESLint.	c07f38fd3f46d40410aae17be39d8f50cc7c60a3
	Fix line length issues in our JavaScript code. We had a number of places where the line length was slightly too long. This updates various doc comments and code to fit within the max line length, satisfying ESLint.	a23e9270205e93a76642fa7092b1157e0bad8589

Diff:

Revision 2 (+170 -102)

Show changes

	djblets/static/djblets/js/datagrid.js
	djblets/static/djblets/js/jquery.gravy.backboneUtils.js
	djblets/static/djblets/js/jquery.gravy.inlineEditor.js
	djblets/static/djblets/js/avatars/views/fileUploadSettingsFormView.es6.js
	djblets/static/djblets/js/configForms/views/listView.es6.js
	djblets/static/djblets/js/extensions/models/extensionHookModel.es6.js
	djblets/static/djblets/js/extensions/views/extensionManagerView.es6.js
	djblets/static/djblets/js/forms/models/conditionChoiceModel.es6.js
	8 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to release-4.x (13bf726)