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

Update at-since up to May 3, 2020 #4705

Merged
merged 3 commits into from
May 3, 2020

Conversation

daniel-beck
Copy link
Member

Ideally merged before 2.235, so we have nice LTS Javadoc.

@@ -55,7 +55,7 @@ public static String encodeTo(Node node) {
}

/**
* @since TODO
* @since 2.230
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -91,7 +91,7 @@ protected void killComputer(Computer c) {
/**
* Get the disabled administrative monitors
*
* @since TODO
* @since 2.230
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -102,7 +102,7 @@ protected void killComputer(Computer c) {
/**
* Set the disabled administrative monitors
*
* @since TODO
* @since 2.230
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -1042,7 +1042,7 @@ private static String get(JSONObject o, String prop) {
/**
* Popularity of this plugin.
*
* @since TODO
* @since 2.233
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -41,7 +41,7 @@ private AgentComputerUtil() {
* separate thread on agents, that'll fail.)
*
* @return null if the calling thread doesn't have any trace of where its master is.
* @since TODO
* @since 2.235
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -133,7 +133,7 @@ public ContextMenu add(String url, String icon, String text, boolean post, boole
/**
* Add a header row (no icon, no URL, rendered in header style).
*
* @since TODO
* @since 2.231
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -276,7 +276,7 @@ public void run(JellyContext context, XMLOutput output) throws JellyTagException

/**
* True to display this item as a section header.
* @since TODO
* @since 2.231
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -5,7 +5,7 @@

The help link is rendered as an SVG with an (?) icon.

@since TODO
@since 2.234
Copy link
Member Author

Choose a reason for hiding this comment

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

*/
@Deprecated // since TODO
@Deprecated
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -1720,7 +1720,7 @@ public static int determineRows(String s) {
*/
@Deprecated
@Restricted(DoNotUse.class)
@RestrictedSince("since TODO")
@RestrictedSince("2.173")
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -115,7 +115,7 @@ protected String trimEOL(String line) {
* Convenience subclass for cases where you wish to process lines being sent to an underlying stream.
* {@link #eol} will typically {@link OutputStream#write(byte[], int, int)} to {@link #out}.
* Flushing or closing the decorated stream will behave properly.
* @since FIXME
* @since 2.173
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -159,7 +159,7 @@

/**
* Use {@link #Slave(String, String, ComputerLauncher)} and set the rest through setters.
* @deprecated since FIXME
* @deprecated since 2.184
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -52,7 +52,7 @@ public AbstractCloudSlave(@NonNull String name, String remoteFS, ComputerLaunche

/**
* Use {@link #AbstractCloudSlave(java.lang.String, java.lang.String, hudson.slaves.ComputerLauncher)}
* @deprecated since FIXME
* @deprecated since 2.184
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -64,7 +64,7 @@ public AbstractCloudSlave(String name, String nodeDescription, String remoteFS,

/**
* Use {@link #AbstractCloudSlave(java.lang.String, java.lang.String, hudson.slaves.ComputerLauncher)}
* @deprecated since FIXME
* @deprecated since 2.184
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -243,7 +243,7 @@ public FormValidation doCheckWebSocket(@QueryParameter boolean webSocket, @Query
* This flag is checked in {@code config.jelly} before displaying the
* Java Web Start button.
* @return {@code true} if Java Web Start button should be displayed.
* @since FIXME
* @since 2.153
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -37,7 +37,7 @@

/**
* Utilities useful when working with various XML types.
* @since 1.596.1 and 1.600, unrestricted since TODO
* @since 1.596.1 and 1.600, unrestricted since 2.179
Copy link
Member Author

Choose a reason for hiding this comment

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

@daniel-beck daniel-beck added the skip-changelog Should not be shown in the changelog label May 3, 2020
@daniel-beck daniel-beck added the ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback label May 3, 2020
@daniel-beck
Copy link
Member Author

Can be merged (once CI passes just to be safe).

Copy link
Member

@oleg-nenashev oleg-nenashev left a comment

Choose a reason for hiding this comment

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

Fine with merge

@timja timja merged commit 40c6728 into jenkinsci:master May 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback skip-changelog Should not be shown in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants