Skip to content

Proposed changes to accompany the SCL service loader change#418

Merged
zbynek merged 4 commits intogwtproject:mainfrom
niloc132:10103-doc-updates
Feb 25, 2026
Merged

Proposed changes to accompany the SCL service loader change#418
zbynek merged 4 commits intogwtproject:mainfrom
niloc132:10103-doc-updates

Conversation

@niloc132
Copy link
Member

No description provided.

The `-noserver` option instructs development mode to not start the embedded Jetty instance. In its place, you would run the J2EE container of your choice and simply use that
in place of the embedded Jetty instance.
GWT provides several ways to customize the development experience to suit your application's needs:
* The simplest and most flexible option is to use CodeServer instead of DevMode, allowing you to run your own application server, and only rely on GWT to
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we use backticks for all Java class names in this paragraph (CodeServer, DevMode, ServiceLoader)?

Copy link
Member Author

Choose a reason for hiding this comment

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

I think I fixed them all, flag anything I missed?

Copy link
Contributor

Choose a reason for hiding this comment

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

I meant to ask whether

is to use CodeServer instead of DevMode

should be

is to use CodeServer instead of DevMode

to emphasize that this refers to Java classes. Checking more closely, DevMode by itself is not using backticks throughough this page, only when used as fully qualified class name. But maybe CodeServer should be also mentioned as FQCN somewhere on the page?

Copy link
Member Author

Choose a reason for hiding this comment

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

com.google.gwt.dev.DevMode is defined later on the page, though CodeServer never is. A link to superdevmode.md on first mention perhaps?

Also, I did wrap them all in backticks... but haven't yet pushed the commit, I'm sorry, was still working on the last point and never concluded it.

Copy link
Contributor

Choose a reason for hiding this comment

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

A link to superdevmode.md on first mention perhaps

Sounds good.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm seeing now that the very next sentence has a link to superdevmode.md... so I'm not sure what makes sense. I've gone ahead and added an anchor to https://www.gwtproject.org/articles/superdevmode.html#Starting so that we at least have CodeServer's class and args in view.

@niloc132 niloc132 marked this pull request as ready for review February 11, 2026 01:45
@niloc132 niloc132 requested a review from zbynek February 25, 2026 21:13
@zbynek zbynek merged commit 09c0a93 into gwtproject:main Feb 25, 2026
1 check passed
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.

2 participants