typelevel / typelevel/cats-parse
Remove isScalaJs/isScalaJvm on minor version update
Nobody has claimed this yet.
- Dominant language
- Scala
- Stars
- 245
- Forks
- 51
- Avg merge
- 3h 20m
- Merged PRs (30d)
- 1
Description
Rel https://github.com/typelevel/cats-parse/pull/384#issuecomment-1059808127
Now that scala.js(https://www.scala-js.org/news/2022/02/14/announcing-scalajs-1.9.0/) and scala native(https://github.com/scala-native/scala-native/pull/2565) support java.util.BitSet, we no longer need these apis. Unfortunately these fields are public, so we will remove them in next minor version update to reduce downstream users' confusion.
Contributor guide
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
Search the repository for isScalaJs and isScalaJvm, then inspect any related tests or usages. Done means removing these public APIs as part of a minor version update and confirming the test suite still passes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- scala
- Domain
- api
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100