-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Trim down supported file extensions on Windows and Linux #183
Trim down supported file extensions on Windows and Linux #183
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@mrclary, could you take a look at this one? Thanks! |
Let me take a look at this. I agree that Spyder should not be the default application, but it should be in the "Open with..." context menu and these files should be openable from a Spyder open-file dialogue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ccordoba12, I still need to test this PR, but at least the trailing commas in lists need to be removed, otherwise JSONDecodeError
results.
fe2b3ff
to
7c8aadb
Compare
@mrclary, this is ready for another review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me; thanks @ccordoba12
Hi! This is the friendly conda-forge automerge bot! I considered the following status checks when analyzing this PR:
Thus the PR was passing and merged! Have a great day! |
Fixes spyder-ide/spyder#22389.