Skip to content

[find file/python] fixed snippet perf issue #37

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

Merged
merged 3 commits into from
Dec 31, 2024

Conversation

majvax
Copy link
Collaborator

@majvax majvax commented Dec 30, 2024

Description

  • Fixed the python "find file snippet" according to pep 471

Type of Change

  • Improvement to existing snippet(s)

Checklist

  • I have tested all submitted code snippets and verified they work as intended
  • My code follows the style guidelines of this project
  • I have commented my code appropriately where needed
  • I have updated the documentation accordingly (if applicable)
  • My changes generate no new warnings
  • For new snippets:
    • I have added appropriate tags
    • I have included a clear description
    • The category is appropriate (or I have created a new one)
    • Code is properly formatted and commented

Copy link

netlify bot commented Dec 30, 2024

Deploy Preview for quicksnip ready!

Name Link
🔨 Latest commit 719605f
🔍 Latest deploy log https://app.netlify.com/sites/quicksnip/deploys/6773d31fca061e000878931c
😎 Deploy Preview https://deploy-preview-37--quicksnip.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@majvax majvax changed the title Added vector manipulation in C++ Added 2 C++ snippet Dec 31, 2024
@majvax majvax changed the title Added 2 C++ snippet [find file/python] fixed snippet perf issue Dec 31, 2024
@Mathys-Gasnier Mathys-Gasnier merged commit f8ff933 into quicksnip-dev:main Dec 31, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants