Take node configuration header private in 8.x

Aperta
#8,631 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
35/100
Tipo di issue
Refactoring
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
cpp
Ambito
api

Direzione di ricerca

Start by reading include/ccf/node/configuration.h and tracing how its public struct is used by applications and the configuration subsystem. The issue proposes making that header private for 8.x, with any needed app-facing configuration exposed through an additive subsystem interface. Done means the public struct is no longer part of the API and required application configuration remains available; the issue names no tests.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Almost all changes to https://github.com/microsoft/CCF/blob/main/include/ccf/node/configuration.h can't be made 100% backwards-compatible, as it's a public header struct change.

However, we often want to change it, while adding new features or deprecating old ones, as in marking them optional or whatever.

I suggest we consider taking this private, as I didn't find a good reason to keep it public.

On demand, we can do additive change to a config subsystem interface to report the needed conf to the app if the app is interested in it in the future.

Lingua principale
C++
Stelle
876
Fork
264
Merge medio
1g 16h
PR unite (30g)
127

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di microsoft/CCF

Tutte le issue di microsoft/CCF

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.