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.

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

refactor!: declare private and public interface #456

Merged
merged 10 commits into from
Aug 23, 2024

Conversation

vdusek
Copy link
Collaborator

@vdusek vdusek commented Aug 22, 2024

Description

  • declare private and public interface

Issues

  • N/A

Testing

  • All existing tests are passing.

Checklist

  • CI passed

@vdusek vdusek added t-tooling Issues with this label are in the ownership of the tooling team. debt Code quality improvement or decrease of technical debt. labels Aug 22, 2024
@vdusek vdusek added this to the 96th sprint - Tooling team milestone Aug 22, 2024
@vdusek vdusek requested a review from janbuchar August 22, 2024 17:40
@vdusek vdusek self-assigned this Aug 22, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Pull Request Tookit has failed!

Pull request is neither linked to an issue or epic nor labeled as adhoc!

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Pull Request Tookit has failed!

Pull request is neither linked to an issue or epic nor labeled as adhoc!

@vdusek vdusek added the adhoc Ad-hoc unplanned task added during the sprint. label Aug 22, 2024
@vdusek vdusek force-pushed the define-private-and-public-modules branch from 35dfde7 to ea3e1de Compare August 23, 2024 11:41
@vdusek vdusek force-pushed the define-private-and-public-modules branch from ea3e1de to f966400 Compare August 23, 2024 13:00
Copy link
Collaborator

@janbuchar janbuchar left a comment

Choose a reason for hiding this comment

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

Awesome!

@vdusek vdusek merged commit d6738df into master Aug 23, 2024
19 checks passed
@vdusek vdusek deleted the define-private-and-public-modules branch August 23, 2024 14:18
vdusek added a commit that referenced this pull request Aug 26, 2024
### Description

- refactor: storage-related models are in base storage client
- basically I just split `crawlee/_models.py` to `crawlee/_request.py`
and `crawlee.base_storage_client._models.py`

### Issues

- Just a follow-up of #456 

### Testing

- Tests are passing

### Checklist

- [x] CI passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adhoc Ad-hoc unplanned task added during the sprint. debt Code quality improvement or decrease of technical debt. t-tooling Issues with this label are in the ownership of the tooling team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants