llnl / llnl/lmt

remove legacy code

Open
#59 0 comments 0 reactions 1 assignee View on GitHub

@defaziogiancarlo is already working on this.

Since Apr 4, 2022.

Dominant language
C
Stars
77
Forks
21
PR merge metrics
No merged PRs in 30d

Description

There's a lot of legacy code that doesn't appear to get used outside of testing it. The code for cerebro modules that no longer exist and old metric protocols could be removed. There are obvious, and probably some subtle, backward-compatibility issues that might prevent this from happening.

Contributor guide

No contributing guide indexed for this repository

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.