Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Support converting '.NET TP Worker' to the numeric thread ID on .NET 8+#103

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@erikmav
Copy link
Contributor

No description provided.

zhiweiv, mdegroux, dp0h, michal-ciechan, and poormonfared reacted with thumbs up emoji
@erikmaverikmavforce-pushed thedev/erikmav/net8ThreadPoolWorker branch fromdff597c toab74a0aCompareJanuary 12, 2024 18:41
@michal-ciechan
Copy link

Any chance this can be merged, so that we don't have to fork.

@FreeAndNil
Copy link
Contributor

@fluffynuts can you take a look?

@vy
Copy link
Member

vy commentedJan 31, 2024

@erikmav,@michal-ciechan,@FreeAndNil, log4net is in great need of maintainers.@fluffynuts, the only active maintainer, and whom I believe will soon take care of this PR, can definitely use some help.Would any of you be willing to step up as a maintainer? If so, could you share this in an email todev@, please? I cannot speak for the rest of the PMC, but IMO, we can grant a volunteer the committer rights. Equipped with that, they can merge PRs, commit changes, etc. The release cannot be done by committers; there PMC members are needed for reviewing and performing the release anyway.

fluffynuts, FreeAndNil, and poormonfared reacted with thumbs up emoji

@fluffynutsfluffynuts merged commit765567a intoapache:masterJan 31, 2024
@fluffynuts
Copy link
Contributor

@michal-ciechan merged - I can't make any guarantees of a release date though, but perhaps after I've finished some other opensource PR I've been working on in the little spare time I have for the last 3 weeks (:

poormonfared reacted with thumbs up emojimichal-ciechan reacted with heart emoji

@poormonfared
Copy link

Hi, when will this fix get released?! It's a really hot fix.
We already had similar issues twice. 1-wrong thread name in updating to.net 6; 2-wrong process name after migrating to.net 7.
Why should we see this again? It would be nice if we carefully tracked our previous issues.

@fluffynuts
Copy link
Contributor

Considering my constrained time, and even though@FreeAndNil recently stepped up to be a maintainer, I don't see any time really soon. You can, in the meantime, check out the repo, build, and distribute a dll with your app if it's that important. It's not convenient, I'll admit, but if it's urgent, it's an option.

@poormonfared
Copy link

@fluffynuts actually, if we replaced ThreadName with our customized property field, then we wouldn't need to replace the library anymore.

fluffynuts and zhiweiv reacted with thumbs up emoji

@FreeAndNilFreeAndNil added this to the2.0.16 milestoneMar 5, 2024
@FreeAndNil
Copy link
Contributor

@erikmav,@michal-ciechan,@poormonfared we've releasedhttps://www.nuget.org/packages/log4net/2.0.16

zhiweiv and poormonfared reacted with thumbs up emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@fluffynutsfluffynutsfluffynuts approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

2.0.16

Development

Successfully merging this pull request may close these issues.

6 participants

@erikmav@michal-ciechan@FreeAndNil@vy@fluffynuts@poormonfared

[8]ページ先頭

©2009-2025 Movatter.jp