SWAT-engineering / SWAT-engineering/java-watch
Reduce visibility of `JDK...` classes
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- Java
- Stars
- 1
- Forks
- 0
- Avg merge
- 14h 40m
- Merged PRs (30d)
- 1
Description
Currently, the JDK... classes in package engineering.swat.watch.impl.jdk are public, but they aren't really intended to be used directly by consumers of the library. Consider reducing their visibility.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by inspecting the JDK... classes in package engineering.swat.watch.impl.jdk and checking how their visibility is used elsewhere in the library. Determine which classes are intended only for internal use and verify that the public API and existing tests still work after reducing visibility.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100