Skip to content

Conversation

gerhardol
Copy link
Member

add .editorconfig from GE to suppress WFO1000
The complete config is included, with many settings.

No usage at all.

Note that with upcoming 6.0.1, you get a popup due to a changed API.
The plugin may work before, but not applying the dark theme
(may be required manually also targetting .net9, all untested).

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

Support GitExtensions 6.0

  • Bugfix
  • Feature
  • Refactoring
  • Build related changes
  • Documentation
  • Other:

What is the current behavior?

Popup and no dark mode, unknown with GE6

What is the new behavior?

No popup but still unknown.

Does this PR introduce a breaking change?

  • Yes
  • No

New .net9, new GE version

Other information

add .editorconfig from GE to suppress WFO1000
The complete config is included, with many settings.
@gerhardol gerhardol requested a review from lekhmanrus August 16, 2025 21:16
README.md Outdated
| 4.0.1 <= v <= 4.0.2 | 2.0.5 |
| 4.1.x <= v <= 4.2.0 | 2.0.6 <= v |
| 4.2.x <= v <= 5.0.0 | 2.0.7 <= v |
| 5.0.x <= v | 3.0.0 <= v |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
| 5.0.x <= v | 3.0.0 <= v |
| 5.0.x <= v <= 6.0.0 | 3.0.0 <= v |

| 4.1.x <= v <= 4.2.0 | 2.0.6 <= v |
| 4.2.x <= v <= 5.0.0 | 2.0.7 <= v |
| 5.0.x <= v | 3.0.0 <= v |
| 6.0.1 <= v | 3.1.0 <= v |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
| 6.0.1 <= v | 3.1.0 <= v |
| 6.0.x <= v | 3.1.0 <= v |

@lekhmanrus
Copy link
Collaborator

Thanks @gerhardol !

Copy link
Member

@RussKie RussKie left a comment

Choose a reason for hiding this comment

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

:shipit:

@gerhardol
Copy link
Member Author

gerhardol commented Aug 17, 2025

Will update GitExtensions.Extensibility to 0.3.3 (likely) too.
gitextensions/gitextensions.extensibility#33

update version of GitExtensions.Extensibility to merged version

add .editorconfig from GE to suppress WFO1000
The complete config is included, with many settings.
@lekhmanrus
Copy link
Collaborator

It looks like GitExtensions.Extensibility version 0.4.0.106 hasn’t been published yet:
https://www.nuget.org/packages/GitExtensions.Extensibility/

@gerhardol
Copy link
Member Author

It looks like GitExtensions.Extensibility version 0.4.0.106 hasn’t been published yet:
https://www.nuget.org/packages/GitExtensions.Extensibility/

The publish failed due to an expired token, waiting for it to be updated.

@maraf
Copy link
Member

maraf commented Aug 26, 2025

Package is now being published https://www.nuget.org/packages/GitExtensions.Extensibility/0.4.0.111

@lekhmanrus
Copy link
Collaborator

Thanks @maraf!
Could somebody retrigger AppVeyor checks please?

@gerhardol
Copy link
Member Author

It builds, but GitExtensions.Extensibility.dll still seem to be ld so not updated.

@gerhardol
Copy link
Member Author

Still not working, some discussion in gitextensions/gitextensions.pluginmanager#82

@gerhardol
Copy link
Member Author

@gitextensions/git-extensions-team This is still a blocker for 6.0.1 - at least one plugin should be OK

@RussKie
Copy link
Member

RussKie commented Sep 10, 2025

Apologies, what's blocked? I thought it was resolved with gitextensions/gitextensions.pluginmanager#82

@gerhardol
Copy link
Member Author

Apologies, what's blocked? I thought it was resolved with gitextensions/gitextensions.pluginmanager#82

The build in this PR uses the old version of GitExtensions.Extensibility

@maraf
Copy link
Member

maraf commented Sep 10, 2025

The PR updates the Extensibility package to 0.4.* and for the assembly, it has instructions to download latest https://github.com/gitextensions/GitExtensions.GerritPlugin/blob/master/src%2FGitExtensions.GerritPlugin%2FGitExtensions.GerritPlugin.csproj.user#L7.

It should be all that is needed. Where do you see old version?

@gerhardol
Copy link
Member Author

It should be all that is needed. Where do you see old version?

When I downloaded the artifact for this PR and put that in UserPlugins, I got the popup about the missing API that should be resolved by the update

@maraf
Copy link
Member

maraf commented Sep 10, 2025

Okay, I can repro it as well

EDIT: It doesn't happen when I build this branch locally
EDIT2: Now it makes sense

  Searching for Git Extensions release 'latest' on 'GitHub'.
  Selected release 'v5.2.1'.
  Selected asset 'GitExtensions-Portable-x64-5.2.1.18061-0d74cfdc3.zip'.
  Downloading 'https://github.com/gitextensions/gitextensions/releases/download/v5.2.1/GitExtensions-Portable-x64-5.2.1.18061-0d74cfdc3.zip'...

@maraf
Copy link
Member

maraf commented Sep 10, 2025

@gerhardol It should be resolved now

@gerhardol
Copy link
Member Author

Thanks @maraf !
There is one PR in the 6.0.1 milestone open still, a release build should be possible soon.
... and all plugins to be updated, with correct dependency ranges ....

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.

4 participants