Skip to content

Adding check of javadoc broken links into travis pull request run #13

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

Closed
wants to merge 1 commit into from

Conversation

jtulach
Copy link
Contributor

@jtulach jtulach commented Jan 25, 2016

Run more tests to ensure documentation doesn't have broken links and missing tags.

@chumer
Copy link
Member

chumer commented Jan 25, 2016

I tried this before, but failed. Unfortunately mx javadoc requires to have "mx build" run before. In my opinion mx javadoc should just do the build automatically if not yet built. What do you think @gilles-duboscq ?

@chrisseaton
Copy link
Contributor

Or you can have multiple commands as in mx/mx build && mx/mx javadoc --unified, or call a shell script with many commands in it.

@chumer
Copy link
Member

chumer commented Jan 25, 2016

mx/mx build && mx/mx javadoc --unified does not work on travis. don't know the reason. stopped investigating

@jtulach
Copy link
Contributor Author

jtulach commented Jan 25, 2016

In the light of #15 this change is obsolete.

@jtulach jtulach closed this Jan 25, 2016
@gilles-duboscq
Copy link
Member

Regarding javadoc: mx javadoc already does the build by default. What was the problem?

@jtulach jtulach deleted the Javadoc4Travis branch January 26, 2016 08:40
dougxc pushed a commit that referenced this pull request Apr 12, 2016
…-construction to master

* commit '9a91ef538a1fd7761e22602d5b21bbb473706bff':
  Fix race condition between Breakpoint construction and access from listeners
zakkak added a commit to zakkak/mandrel that referenced this pull request Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants