Skip to content

Fix broken links, typos, and minor code quality issues#2118

Open
apoorvdarshan wants to merge 1 commit into
Netflix:masterfrom
apoorvdarshan:fix/broken-links-typos-code-quality
Open

Fix broken links, typos, and minor code quality issues#2118
apoorvdarshan wants to merge 1 commit into
Netflix:masterfrom
apoorvdarshan:fix/broken-links-typos-code-quality

Conversation

@apoorvdarshan

Copy link
Copy Markdown

Summary

  • Fix all broken Javadoc links: netflix.github.comnetflix.github.io across READMEs, build.gradle, and Clojure source (GitHub Pages domain migration)
  • Replace stale Travis CI badge with GitHub Actions CI badge (project migrated to GitHub Actions in Update Github Actions to use latest NetflixOSS recommendations #2115)
  • Fix variable name typo: springframeworkVesionspringframeworkVersion in hystrix-javanica/build.gradle
  • Fix test class name typo: HystricsMetricsControllerTestHystrixMetricsControllerTest
  • Replace StringBuffer with StringBuilder in HystrixCommandCompletion.toString() (thread safety not needed, standard Java best practice)

Files changed (12)

Category Files
Javadoc links README.md, hystrix-core/README.md, hystrix-core/build.gradle, hystrix-contrib/README.md, hystrix-contrib/hystrix-codahale-metrics-publisher/README.md, hystrix-contrib/hystrix-servo-metrics-publisher/README.md, hystrix-contrib/hystrix-yammer-metrics-publisher/README.md, hystrix-contrib/hystrix-request-servlet/README.md, hystrix-contrib/hystrix-clj/src/main/clojure/com/netflix/hystrix/core.clj
CI badge README.md
Build typo hystrix-contrib/hystrix-javanica/build.gradle
Test class rename hystrix-contrib/hystrix-metrics-event-stream-jaxrs/.../HystrixMetricsControllerTest.java
Code quality hystrix-core/.../HystrixCommandCompletion.java

Test plan

  • All changes are low-risk: documentation fixes, typo corrections, and a standard Java best-practice replacement
  • No behavioral changes — build variable rename is internal-only, test class rename is test-only
  • CI should pass without issues

- Fix all Javadoc links: netflix.github.com → netflix.github.io (GitHub Pages domain migration)
- Replace stale Travis CI badge with GitHub Actions CI badge
- Fix variable name typo: springframeworkVesion → springframeworkVersion in hystrix-javanica/build.gradle
- Fix test class name typo: HystricsMetricsControllerTest → HystrixMetricsControllerTest
- Replace StringBuffer with StringBuilder in HystrixCommandCompletion.toString()
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.

1 participant