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

Split profile classes into their own separate files #282

Merged
merged 1 commit into from
May 29, 2024

Conversation

amercader
Copy link
Member

The old profiles.py file was starting to become unwieldy. This messes up history but I think it's worth doing, considering there will be more profiles coming in the future. Also took the opportunity to format the new files with black

The old profiles.py file was starting to become unwieldy. This messes up
history but I think it's worth doing, considering there will be more
profiles coming in the future
@amercader amercader mentioned this pull request May 23, 2024
@amercader
Copy link
Member Author

Note: failures with master are caused by an issue with the responses package: getsentry/responses#708, there should be a release soon: getsentry/responses#714

@amercader amercader merged commit 373cb77 into master May 29, 2024
6 of 8 checks passed
@amercader amercader deleted the split-profile-files branch May 29, 2024 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant