Remove company property

[Bilbottom]

Summary by Sourcery

Remove the 'company' property from test files, schemas, and SQL models to streamline data models and update the pre-commit configuration to exclude all files from SQL linting temporarily.

Enhancements:

• Remove the 'company' property from various test files and schemas to simplify data models.

Build:

• Update pre-commit configuration to exclude all files from SQL linting temporarily.

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request removes the 'company' property from various test files, schema definitions, and SQL select statements. The changes ensure that the 'company' field is no longer part of the data model, reflecting a shift in how company information is managed. Additionally, the pre-commit configuration has been updated to exclude all files from sqlfluff-lint temporarily.

File-Level Changes

Files	Changes
billiam_database/models/presentation/tests/test__pl__daily_tracker.yml billiam_database/models/staging/tests/test__stg__daily_tracker.yml billiam_database/models/presentation/tests/test__pl__daily_metrics.yml billiam_database/models/presentation/tests/test__pl__task_details.yml	Removed 'company' property from input and expect rows in various test files.
billiam_database/models/presentation/_schema.yml billiam_database/models/staging/_schema.yml	Removed 'company' field definition and constraints from schema files.
billiam_database/models/presentation/pl__daily_tracker.sql billiam_database/models/staging/stg__daily_tracker.sql	Removed 'company' column from final select statements in SQL files.

---

Tips

• Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
• Continue your discussion with Sourcery by replying directly to review comments.
• You can change your review settings at any time by accessing your dashboard:
  • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
  • Change the review language;
• You can always contact us if you have any questions or feedback.

[sourcery-ai]

Hey @Bilbottom - I've reviewed your changes - here's some feedback:

Overall Comments:

• Could you provide some context on why the 'company' property is being removed? Understanding the rationale behind this change would be helpful, especially considering its impact across multiple parts of the data model.
• We noticed that SQL linting has been excluded in the .pre-commit-config.yaml file. Is this a temporary measure? If so, it would be good to have a plan or timeline for re-enabling it in the future.

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}