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

Exclude obsidian.md preferences #6306

Merged
merged 3 commits into from Mar 7, 2023
Merged

Conversation

sanjarcode
Copy link
Contributor

@sanjarcode sanjarcode commented Mar 4, 2023

Ignore .obsidian folder for stats, by adding it as a path exclusion.

Description

When using https://obsidian.md/, a .obsidian folder is created inside the project (notes in this case) folder. It contains the plugins' source code, which of course, is not the focus of notes.

Case - where this is happening

Checklist:

  • I am adding new or changing current functionality
    • I have added or updated the tests for the new or changed functionality.

When using https://obsidian.md/, a `.obsidian` folder is created inside the project (notes in this case) folder. It contains the plugins' source code, which of course, is not the focus of notes.
@sanjarcode sanjarcode requested a review from a team as a code owner March 4, 2023 15:07
Copy link
Member

@lildude lildude left a comment

Choose a reason for hiding this comment

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

Please add to the tests here.

@lildude
Copy link
Member

lildude commented Mar 6, 2023

Please add to the tests here.

NM. I've done it for you 😁

@sanjarcode
Copy link
Contributor Author

sanjarcode commented Mar 6, 2023

Thanks a lot, @lildude

BTW, how much time before I can preview/live-test if this works?

@lildude
Copy link
Member

lildude commented Mar 7, 2023

You won't see this take effect until I make and deploy the next release which I'm assuming to do in the next week or two.

@lildude lildude merged commit 0578636 into github-linguist:master Mar 7, 2023
zekigedir referenced this pull request Mar 12, 2023
* Add .cnf to INI

* Add openssl.cnf

* Increase acceptable errors

---------

Co-authored-by: Colin Seymour <colin@github.com>
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.

None yet

2 participants