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

Clean deletion of analytic_tag_id setting parameter and pre-commit cleanings #236

Merged
merged 2 commits into from
Jul 11, 2024

Conversation

NoeBerdoz
Copy link
Contributor

@NoeBerdoz NoeBerdoz commented Jul 10, 2024

  • Delete dead files that weren't deleted and are related to this PR #234

  • Make the repository compliant with pre-commit

@NoeBerdoz NoeBerdoz changed the title Clean deletion of analytic_tag_id setting parameter Clean deletion of analytic_tag_id setting parameter and pre-commit cleanings Jul 10, 2024
@NoeBerdoz NoeBerdoz requested a review from ecino July 10, 2024 11:04
@ecino
Copy link
Member

ecino commented Jul 11, 2024

Sorry, I also applied pre-commit with another PR. Can you please rebase and squash your commits (one single commit is enough I think).

@NoeBerdoz NoeBerdoz force-pushed the 14-clean-delete-and-precommit branch from 645e0da to daae4db Compare July 11, 2024 13:22
@NoeBerdoz
Copy link
Contributor Author

NoeBerdoz commented Jul 11, 2024

@ecino I squashed my commits, and merged the missing corrections for the pre-commit, PR is ready to be merged

@NoeBerdoz NoeBerdoz merged commit c4430b9 into CompassionCH:14.0 Jul 11, 2024
1 check passed
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.

2 participants