Skip to content

Extract excludes into a file, fix the link checker by adding http://site.icu-project.org/.#1189

Merged
dblock merged 4 commits intoopensearch-project:mainfrom
dblock:lycheeignore
Aug 31, 2021
Merged

Extract excludes into a file, fix the link checker by adding http://site.icu-project.org/.#1189
dblock merged 4 commits intoopensearch-project:mainfrom
dblock:lycheeignore

Conversation

@dblock
Copy link
Member

@dblock dblock commented Aug 31, 2021

Signed-off-by: dblock dblock@dblock.org

Description

  • Fixes the link checker failure we're seeing on main by excluding http://site.icu-project.org/ that is currently MIA.
  • Extract --exclude config into a file.
  • Remove now working or obsolete links from the excludes.

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Wrapper Validation success 15ad48d58c1837a11f7522e3ee4d1da0ec322927

Signed-off-by: dblock <dblock@dblock.org>
@opensearch-ci-bot
Copy link
Collaborator

✅   DCO Check Passed 15ad48d58c1837a11f7522e3ee4d1da0ec322927

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Wrapper Validation success b0fac08

@opensearch-ci-bot
Copy link
Collaborator

✅   DCO Check Passed b0fac08

Signed-off-by: dblock <dblock@dblock.org>
@opensearch-ci-bot
Copy link
Collaborator

✅   DCO Check Passed e02e7cc

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Wrapper Validation success e02e7cc

@opensearch-ci-bot
Copy link
Collaborator

✅   DCO Check Passed e21844383594c37481cdbca320d981bd9b3544a4

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Wrapper Validation success e21844383594c37481cdbca320d981bd9b3544a4

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Precommit success 15ad48d58c1837a11f7522e3ee4d1da0ec322927

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Precommit success b0fac08

@dblock dblock changed the title Extract excludes into a file. Extract excludes into a file, fix the build. Aug 31, 2021
@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Precommit success e02e7cc

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Wrapper Validation success 6e813f92fc6528262bcba50401b9d74dc334c825

@opensearch-ci-bot
Copy link
Collaborator

✅   DCO Check Passed 6e813f92fc6528262bcba50401b9d74dc334c825

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Wrapper Validation success ddafcf740f83878aeb4593a1b3f6fa751a70d6b6

@opensearch-ci-bot
Copy link
Collaborator

✅   DCO Check Passed ddafcf740f83878aeb4593a1b3f6fa751a70d6b6

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Precommit success e21844383594c37481cdbca320d981bd9b3544a4

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Precommit success 6e813f92fc6528262bcba50401b9d74dc334c825

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Wrapper Validation success 733c4363fccaf268b042a3c348aaef16fe043d28

@opensearch-ci-bot
Copy link
Collaborator

✅   DCO Check Passed 733c4363fccaf268b042a3c348aaef16fe043d28

Signed-off-by: dblock <dblock@dblock.org>
@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Wrapper Validation success 708ca10

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Wrapper Validation success 23957639e910475315ed85cc585930b3791a5282

Copy link
Contributor

@VachaShah VachaShah Aug 31, 2021

Choose a reason for hiding this comment

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

--exclude-mail is just a flag to exclude checking mailing addresses. The "**/*.html" "**/*.md" "**/*.txt" "**/*.json" are the type of files that need to be checked. The file can be renamed to reflect that.

Copy link
Member Author

Choose a reason for hiding this comment

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

🤦 my bad

@opensearch-ci-bot
Copy link
Collaborator

✅   DCO Check Passed 23957639e910475315ed85cc585930b3791a5282

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Wrapper Validation success 453d64ceeb25af15d7cf2ced912e05657169002d

@opensearch-ci-bot
Copy link
Collaborator

✅   DCO Check Passed 453d64ceeb25af15d7cf2ced912e05657169002d

@VachaShah VachaShah self-requested a review August 31, 2021 20:24
@nknize
Copy link
Contributor

nknize commented Aug 31, 2021

Not sure what lychee is but is this addressing the ongoing linkchecker failure?

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Precommit success 23957639e910475315ed85cc585930b3791a5282

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Precommit success 453d64ceeb25af15d7cf2ced912e05657169002d

@dblock
Copy link
Member Author

dblock commented Aug 31, 2021

Not sure what lychee is but is this addressing the ongoing linkchecker failure?

It's the link checker. One of the websites, http://site.icu-project.org/, has gone MIA, so it started failing. This addresses that + makes it easier to address it in the future by extracting the configuration into config files.

@dblock
Copy link
Member Author

dblock commented Aug 31, 2021

start gradle check

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 708ca10
Log 451

Reports 451

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 453d64ceeb25af15d7cf2ced912e05657169002d
Log 453

Reports 453

Signed-off-by: dblock <dblock@dblock.org>
@dblock dblock changed the title Extract excludes into a file, fix the build. Extract excludes into a file, fix the build by adding http://site.icu-project.org/. Aug 31, 2021
@dblock dblock changed the title Extract excludes into a file, fix the build by adding http://site.icu-project.org/. Extract excludes into a file, fix the link checker by adding http://site.icu-project.org/. Aug 31, 2021
@dblock
Copy link
Member Author

dblock commented Aug 31, 2021

start gradle check

@opensearch-ci-bot
Copy link
Collaborator

✅   DCO Check Passed e3382e6

@dblock
Copy link
Member Author

dblock commented Aug 31, 2021

Can I please have another CR @VachaShah after fixing my epic fail of --exclude-mail:) And thank you for catching that.

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Wrapper Validation success e3382e6

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Precommit success e3382e6

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success e3382e6
Log 454

Reports 454

@dblock dblock merged commit f298a41 into opensearch-project:main Aug 31, 2021
@dblock dblock deleted the lycheeignore branch August 31, 2021 22:21
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.

6 participants