Skip to content

YETUS-1271. Fix javadoc build#383

Merged
aw-was-here merged 4 commits into
apache:mainfrom
effectivemachines:yetus-1271-javadoc
Apr 30, 2026
Merged

YETUS-1271. Fix javadoc build#383
aw-was-here merged 4 commits into
apache:mainfrom
effectivemachines:yetus-1271-javadoc

Conversation

@aw-was-here
Copy link
Copy Markdown
Contributor

No description provided.

@aw-was-here
Copy link
Copy Markdown
Contributor Author

I think I'm still getting a -1 because the main branch is failing. That feels like a bug but I know there was a reason for it a long time ago. haha.

Copy link
Copy Markdown
Member

@ndimiduk ndimiduk left a comment

Choose a reason for hiding this comment

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

I guess we don't have a 1.0 release yet, so we're okay to make breaking changes. Would be good to have a release note.

}

/** The doclet environment processor. */
private DocletEnvironmentProcessor processor =
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

FYI, making this private is a compatibility-breaking change.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oops ok yes let me revert that bit.

Comment thread pom.xml
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why the pom changes?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

A lot of this is fallout from upgrading the apache pom which sets a bunch of stuff that it didn't before.

@ndimiduk
Copy link
Copy Markdown
Member

I think I'm still getting a -1 because the main branch is failing. That feels like a bug but I know there was a reason for it a long time ago. haha.

Yeah that behavior never sat well with me. Seems like, if master is failing a check and the branch fixes that check, that should be overall +1.

@aw-was-here aw-was-here merged commit c4ba327 into apache:main Apr 30, 2026
2 of 3 checks 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