PR: Split all plugins into their own modules #7725
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
PR: Split Online Help module
Conflicts: - spyder/app/tests/test_mainwindow.py - spyder/plugins/editor/tests/test_plugin.py - spyder/plugins/ipythonconsole/tests/test_ipythonconsole.py - spyder/plugins/ipythonconsole/widgets/client.py
…te-3 PR: Update split-plugins branch with master (take 3)
PR: Final update to split-plugins
17 tasks
|
@ccordoba12 If you are moving the files anyway, you are harmonizing them all to LF newlines right? |
|
It'll break merging with 3.x, so no, sorry. |
|
Oh, I thought it would do that anyway because of the moves so the LF change wouldn't make things worse, but I guess not. Maybe in the final stages of the 4.0 release process when all the non-trivial PRs are in and the 3.x branch is frozen we can do it for everything, and harmonize the copyright headers at the same time (to be in a standard format, with a beginning copyright year and refined text)? If not, it would have to be for Spyder 5 or after ofc. |
9 tasks
|
Ok, this is going in! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #4591.
With this PR all our internal plugins will have the same structure as the ones developed outside Spyder (e.g.
spyder-terminal,spyder-notebook, etc).We hope this will help outside contributors to better understand how Spyder works and simplify the development of each plugin too.