Removed unused commons fileupload wrapper. by mbien · Pull Request #9121 · apache/netbeans (original) (raw)

Skip to content

Navigation Menu

Sign in

Appearance settings

• • AI CODE CREATION
    * GitHub CopilotWrite better code with AI
    * GitHub SparkBuild and deploy intelligent apps
    * GitHub ModelsManage and compare prompts
    * MCP RegistryNewIntegrate external tools
  • DEVELOPER WORKFLOWS
    * ActionsAutomate any workflow
    * CodespacesInstant dev environments
    * IssuesPlan and track work
    * Code ReviewManage code changes
  • APPLICATION SECURITY
    * GitHub Advanced SecurityFind and fix vulnerabilities
    * Code securitySecure your code as you build
    * Secret protectionStop leaks before they start
  • EXPLORE
    * Why GitHub
    * Documentation
    * Blog
    * Changelog
    * Marketplace

View all features

• • BY COMPANY SIZE
    * Enterprises
    * Small and medium teams
    * Startups
    * Nonprofits
  • BY USE CASE
    * App Modernization
    * DevSecOps
    * DevOps
    * CI/CD
    * View all use cases
  • BY INDUSTRY
    * Healthcare
    * Financial services
    * Manufacturing
    * Government
    * View all industries

View all solutions

• • EXPLORE BY TOPIC
    * AI
    * Software Development
    * DevOps
    * Security
    * View all topics
  • EXPLORE BY TYPE
    * Customer stories
    * Events & webinars
    * Ebooks & reports
    * Business insights
    * GitHub Skills
  • SUPPORT & SERVICES
    * Documentation
    * Customer support
    * Community forum
    * Trust center
    * Partners

View all resources

• • COMMUNITY
    * GitHub SponsorsFund open source developers
  • PROGRAMS
    * Security Lab
    * Maintainer Community
    * Accelerator
    * GitHub Stars
    * Archive Program
  • REPOSITORIES
    * Topics
    * Trending
    * Collections
• • ENTERPRISE SOLUTIONS
    * Enterprise platformAI-powered developer platform
  • AVAILABLE ADD-ONS
    * GitHub Advanced SecurityEnterprise-grade security features
    * Copilot for BusinessEnterprise-grade AI features
    * Premium SupportEnterprise-grade 24/7 support
• Pricing

Provide feedback

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

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Sign in

Sign up

Appearance settings

apache / netbeans Public

• Notifications You must be signed in to change notification settings

• Fork921

• Star 3k

• Code

• Issues 916

• Pull requests 150

• Discussions

• Actions

• Projects

• Security and quality

• Insights

Additional navigation options

Merged

mbien merged 1 commit intoapache:masterfrom

mbien:remove-fileupload

Jan 11, 2026

ConversationCommits (1)ChecksFiles changed

Merged

Removed unused commons fileupload wrapper.#9121

mbien merged 1 commit intoapache:masterfrom

mbien:remove-fileupload

Conversation

Copy link Copy Markdown

Member

mbien commented

Jan 3, 2026

•

edited

Loading

also removed commons_net dependency declaration from jenkins module

couldn't find any usage, potentially a leftover from #9051

`[Remove unused commons fileupload wrapper.](/apache/netbeans/pull/9121/commits/f0db87ce92062c6e7d35226ca6bd03e3ddf5a1ba "Remove unused commons fileupload wrapper.

also removed commons_net dependency declaration from jenkins module") `

[f0db87c](/apache/netbeans/pull/9121/commits/f0db87ce92062c6e7d35226ca6bd03e3ddf5a1ba)

also removed commons_net dependency declaration from jenkins module

mbien added this to the NB29 milestone

Jan 3, 2026

mbien added Code cleanup

Label for cleanup done on the Netbeans IDE

Upgrade Library

Library (Dependency) Upgrade

ci:all-tests

[ci] enable all tests

ci:dev-build

[ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page)

enterprise

[ci] enable enterprise job

and removed ci:all-tests

[ci] enable all tests

labels

Jan 3, 2026

mbien marked this pull request as ready for review

January 3, 2026 03:28

mbien requested review fromlkishalmi, matthiasblaesing and neilcsmith-net

January 11, 2026 01:25

matthiasblaesing approved these changes Jan 11, 2026

View reviewed changes

Copy link Copy Markdown

Contributor

matthiasblaesing left a comment

Choose a reason for hiding this comment

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

Looks sane to me.

lkishalmi approved these changes Jan 11, 2026

View reviewed changes

Copy link Copy Markdown

Contributor

lkishalmi left a comment

Choose a reason for hiding this comment

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

Thank you!

Copy link Copy Markdown

Member Author

mbien commented

Jan 11, 2026

cool, thanks for the reviews!

mbien merged commit ae6273d into apache:master

Jan 11, 2026

79 of 80 checks passed

This file contains hidden or 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

[ Show hidden characters]({{ revealButtonHref }})

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

Reviewers

lkishalmi lkishalmi approved these changes

matthiasblaesing matthiasblaesing approved these changes

neilcsmith-net Awaiting requested review from neilcsmith-net

Assignees

No one assigned

Labels

ci:dev-build

[ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page)

Code cleanup

Label for cleanup done on the Netbeans IDE

enterprise

[ci] enable enterprise job

Upgrade Library

Library (Dependency) Upgrade

Projects

None yet

Milestone

NB29

Development

Successfully merging this pull request may close these issues.

3 participants