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

fix: remove code that deletes config #370

Merged
merged 1 commit into from
Mar 12, 2024

Conversation

hemz10
Copy link
Contributor

@hemz10 hemz10 commented Mar 12, 2024

Description:

Commit Message

fix: remove code that deletes config

see the guidelines for commit messages.

Checklist:

  • I have performed a self-review of my own code
  • I have documented my code in accordance with the documentation guidelines
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the unit tests
  • I only have one commit (if not, squash them into one commit).
  • I have a descriptive commit message that adheres to the commit message guidelines

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 12.40%. Comparing base (b147810) to head (be78c0e).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #370   +/-   ##
=======================================
  Coverage   12.40%   12.40%           
=======================================
  Files          11       11           
  Lines        1177     1177           
=======================================
  Hits          146      146           
  Misses       1019     1019           
  Partials       12       12           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@shreyasbhat0 shreyasbhat0 merged commit e26a1eb into main Mar 12, 2024
6 checks passed
@shreyasbhat0 shreyasbhat0 deleted the fix/test-script-removes-sample-configs branch March 12, 2024 06:29
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.

3 participants