JuliaCollections / JuliaCollections/FunctionalCollections.jl
Change pvec constructor
Open
Nobody has claimed this yet.
- Dominant language
- Julia
- Stars
- 126
- Forks
- 33
- PR merge metrics
- No merged PRs in 30d
Description
As mentioned in the TODO in the readme that the pvec([1,2,3]) needs to be changed to pvec(1,2,3).I have a question whether the constructor needs to be changed i.e. the old constructor should be removed or is to be retained?
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
Read the TODO in the README and inspect the existing pvec constructor. Determine whether pvec(1,2,3) should replace or coexist with pvec([1,2,3]); done means the chosen constructor behavior is implemented and the README no longer describes the old TODO.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- julia
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100