Skip to content

ruff#2882

Merged
JacobCoffee merged 15 commits intomainfrom
ruff
Feb 18, 2026
Merged

ruff#2882
JacobCoffee merged 15 commits intomainfrom
ruff

Conversation

@JacobCoffee
Copy link
Member

Description

  • Tries to bring linting and code formatting since we don't really use anything atm

Copilot AI review requested due to automatic review settings February 6, 2026 15:14
@JacobCoffee JacobCoffee changed the title ruf ruff Feb 6, 2026
@JacobCoffee
Copy link
Member Author

will need a .git-blame-ignore-revs file for this to not make a mess of history

This comment was marked as outdated.

Copilot AI review requested due to automatic review settings February 6, 2026 15:26

This comment was marked as outdated.

Copilot AI review requested due to automatic review settings February 6, 2026 15:37

This comment was marked as outdated.

Copilot AI review requested due to automatic review settings February 6, 2026 15:52

This comment was marked as outdated.

JacobCoffee and others added 15 commits February 6, 2026 09:55
…s import

- Restore null=True on companies.Company contact/email/url fields to avoid
  unintended schema change (noqa DJ001 since removal needs migration)
- Guard against None return from fix_image() in fix_success_story_images
- Re-add jobs.listeners import in JobsAppConfig.ready() for signal registration
- Prefix unused unpacked variables with underscore for RUF059

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…nges

Ruff DJ001 removed null=True from ~30 string-based model fields, but
this changes the DB schema and requires migrations + data handling.
Restore null=True with noqa: DJ001 to keep this as a formatting-only PR.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- cms/tests.py: add tzinfo=datetime.UTC to satisfy DTZ001, update assertion
- users/apps.py: restore `import users.listeners` (signal for auth token
  auto-creation removed by ruff F401) - fixes all API test 401s/errors

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 6, 2026 15:56

This comment was marked as outdated.

@JacobCoffee JacobCoffee requested a review from hugovk February 6, 2026 16:14
@JacobCoffee
Copy link
Member Author

lets try it out, can always revert or adjust.

@JacobCoffee JacobCoffee merged commit 2cf268f into main Feb 18, 2026
10 checks passed
@JacobCoffee JacobCoffee deleted the ruff branch February 18, 2026 01:30
JacobCoffee added a commit that referenced this pull request Feb 18, 2026
Tells git blame to skip past the ruff formatting (#2882) and app
restructure (#2883) commits so blame shows the actual author of each
line.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@JacobCoffee JacobCoffee mentioned this pull request Feb 18, 2026
1 task
JacobCoffee added a commit that referenced this pull request Feb 18, 2026
Add .git-blame-ignore-revs for bulk formatting/restructure commits

Tells git blame to skip past the ruff formatting (#2882) and app
restructure (#2883) commits so blame shows the actual author of each
line.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants