Summary

Add stubs for remaining existing resource subclasses.

Review Request #14338 — Created Feb. 10, 2025 and submitted March 20, 2025, 8:38 a.m.

Information

Owner

david

Repository

RBTools

Branch

master

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

This change adds in API stubs for the other resource subclasses which
currently exist.

Testing Done

Ran unit tests.

Commits

Summary	ID
Add stubs for remaining existing resource subclasses. This change adds in API stubs for the other resource subclasses which currently exist. Testing Done: Ran unit tests.	89a5b0072c1ad09e1f60078ed3f4d351c482acf9

Issues

Description	From	Last Updated
Missing the "Instance variables" header.	chipx86	March 16, 2025, 12:22 p.m.
Missing the "Instance variables" header.	chipx86	March 16, 2025, 12:22 p.m.
Missing the "Instance variables" header.	chipx86	March 16, 2025, 12:22 p.m.
Missing the "Instance variables" header.	chipx86	March 16, 2025, 12:22 p.m.
Missing the "Instance variables" header.	chipx86	March 16, 2025, 12:49 p.m.
Missing the "Instance variables" header.	chipx86	March 16, 2025, 12:49 p.m.
Missing the "Instance variables" header.	chipx86	March 16, 2025, 12:50 p.m.

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Change looks good. Just a trivial missing bit common to some of the classes.

rbtools/api/resource/diff_file_attachment.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Missing the "Instance variables" header.
```

rbtools/api/resource/draft_file_attachment.py (Diff revision 1)

The issue has been resolved. Show all issues

Missing the "Instance variables" header.

rbtools/api/resource/draft_screenshot.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Missing the "Instance variables" header.
```

rbtools/api/resource/file_attachment.py (Diff revision 1)

The issue has been resolved. Show all issues

Missing the "Instance variables" header.

rbtools/api/resource/file_attachment.py (Diff revision 1)

The issue has been resolved. Show all issues

Missing the "Instance variables" header.

rbtools/api/resource/file_diff.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Missing the "Instance variables" header.
```

rbtools/api/resource/screenshot.py (Diff revision 1)

The issue has been resolved. Show all issues

Missing the "Instance variables" header.

Commits:

	Summary	ID
	Add stubs for remaining existing resource subclasses. This change adds in API stubs for the other resource subclasses which currently exist. Testing Done: Ran unit tests.	abd7cab171b4f05f4d6e41a998cc7c5d3bb2ab9c
	Add stubs for remaining existing resource subclasses. This change adds in API stubs for the other resource subclasses which currently exist. Testing Done: Ran unit tests.	89a5b0072c1ad09e1f60078ed3f4d351c482acf9

Diff:

Revision 2 (+1298 -28)

Show changes

	rbtools/api/resource/base_diff_commit.py
	rbtools/api/resource/diff.py
	rbtools/api/resource/diff_commit.py
	rbtools/api/resource/diff_file_attachment.py
	rbtools/api/resource/draft_diff_commit.py
	rbtools/api/resource/draft_file_attachment.py
	rbtools/api/resource/draft_screenshot.py
	rbtools/api/resource/file_attachment.py
	2 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to master (d16c3c0)