onflow / onflow/atree

Evaluate adding support for atree.OrderedSet

Open
#301 0 comments 1 reaction 1 assignee View on GitHub

@fxamacker is already working on this.

Since Apr 24, 2023.

enhancement
Dominant language
Go
Stars
43
Forks
20
PR merge metrics
No merged PRs in 30d

Description

Issue To Be Solved

Maybe Atree should provide an OrderedSet.

See Cadence PR https://github.com/onflow/cadence/pull/2461

Suggested Solution

Evaluate adding atree.OrderedSet.

One approach is to provide a wrapper around existing atree.OrderedMap in order to reuse existing code. Also consider other approaches depending on priorities for qualities, tradeoffs, and other considerations.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.