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

Add plugins' compatibility with Notepad++ versions in json file for uncompatible plugins #422

Closed
donho opened this issue Mar 5, 2022 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@donho
Copy link
Member

donho commented Mar 5, 2022

Notepad++ has added the ability that Plugins Admin is aware of minimum and maximum Notepad++ version a plugin can deal with (notepad-plus-plus/notepad-plus-plus@a06b404).

For the next release of Notepad++'s stability, the compatibility info should be added in the plugin list.
The new attribute npp-compatible-versions should be added for the incompatible plugin to the latest version of Notepad++ (v8.3.2), regarding notepad-plus-plus/notepad-plus-plus#11198 (or other info).

@donho
Copy link
Member Author

donho commented Mar 5, 2022

@rdipardo
Are you interested in doing this issue?

@chcg chcg added the enhancement New feature or request label Mar 5, 2022
@rdipardo
Copy link
Contributor

rdipardo commented Mar 5, 2022

@rdipardo Are you interested in doing this issue?

If you mean translating the data at notepad-plus-plus/notepad-plus-plus#11198 into JSON, then sure.

Beyond that, you're talking about enough work for several people. The version limits are just suggestions until they're validated with testing. How many "milestone" releases need to be covered? I know 8.3(.1) is top priority for now, but 7.7 broke plugins too, and Scintilla 5 is coming with its own bag of new tricks. 7.9.3 dropped Windows XP; are plugins that haven't updated since 7.9.2 guaranteed compatible? (Not if they call into Comdlg32.dll.)

Once there's a firm list of critical N++ releases and their breaking changes, each plugin would need testing with the version before and after. Something non-interactive like the Win32::Mechanize::NotepadPlusPlus unit test suite is ideal; but that depends on @pryrt getting it to work with N++ 8.3+.

@donho
Copy link
Member Author

donho commented Mar 5, 2022

@rdipardo

If you mean translating the data at notepad-plus-plus/notepad-plus-plus#11198 into JSON, then sure.

Yes, that's what I mean exactly.

Beyond that, you're talking about enough work for several people. The version limits are just suggestions until they're #250. How many "milestone" releases need to be covered? I know 8.3(.1) is top priority for now, but notepad-plus-plus/notepad-plus-plus#5741 (comment) broke plugins too, and Scintilla 5 is coming with its own bag of new tricks. 7.9.3 notepad-plus-plus/notepad-plus-plus#9378 (comment); are plugins that haven't updated since 7.9.2 guaranteed compatible? (Not if they Hsilgos/nppsaveasadmin#24.)

I see your point and agree with you. However, what we've discovered for the moment (with tests from the community and users' bug reports) are enough IMO. The others features regarding the issue coming soon (or not at all) depends on the contributers' (including me) time, and therefore no milestone about it.

Thank you for your help :)

@pryrt
Copy link

pryrt commented Mar 7, 2022

Once there's a firm list of critical N++ releases and their breaking changes,

I think you listed most of them: v7.6.3 (the first stable release after the plugins hierarchy change), v7.7 (scintilla 4.1.4), v7.8 (scintilla 4.2.0), v7.9.3 (for XP issues), v7.9.4 (scintilla 4.4.6), and v8.3 (64bit range indexes).

Something non-interactive like the Win32::Mechanize::NotepadPlusPlus unit test suite is ideal; but that depends on @pryrt pryrt/Win32-Mechanize-NotepadPlusPlus#84.

I had forgotten I hadn't finished the update for that yet; thanks for the reminder. I fixed the test suite and released v0.009 over the weekend.

@donho donho closed this as completed in 5ff20d2 Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants