Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use HasTextEnvelope instances to fill description fields in JSON of key files? #429

Open
smelc opened this issue Nov 2, 2023 · 14 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@smelc
Copy link
Contributor

smelc commented Nov 2, 2023

Maybe we could use HasTextEnvelope instances to provide descriptions for the file contents? Or do we foresee cases where we could have different descriptions for the same type?

Originally posted by @carbolymer in #416 (comment)

@smelc smelc self-assigned this Nov 2, 2023
@smelc
Copy link
Contributor Author

smelc commented Nov 2, 2023

Ooops, realized after the fact that this should have been in cardano-api. It's OK.

Copy link

github-actions bot commented Dec 3, 2023

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Dec 3, 2023
@smelc smelc removed the Stale label Dec 4, 2023
Copy link

github-actions bot commented Jan 4, 2024

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Jan 4, 2024
@smelc smelc removed the Stale label Jan 5, 2024
Copy link

github-actions bot commented Feb 5, 2024

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Feb 5, 2024
@smelc smelc added good first issue Good for newcomers and removed Stale labels Feb 5, 2024
Copy link

github-actions bot commented Mar 7, 2024

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Mar 7, 2024
@carbolymer carbolymer removed the Stale label Mar 7, 2024
@smelc
Copy link
Contributor Author

smelc commented Mar 18, 2024

Witnessing, when working on #651, that it would be relevant for key verification-key to generate non-empty description fields, i.e. to not pass Nothing here: https://github.com/IntersectMBO/cardano-cli/blob/main/cardano-cli/src/Cardano/CLI/EraBased/Run/Key.hs#L111

@smelc smelc changed the title Use HasTextEnvelope instances to provide descriptions for the file contents? Or do we foresee cases where we could have different descriptions for the same type? Use HasTextEnvelope instances to fill description fields in JSON of key files? Mar 18, 2024
Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Apr 18, 2024
@smelc smelc removed the Stale label Apr 19, 2024
Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Jun 20, 2024
@smelc smelc removed the Stale label Jun 20, 2024
Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Jul 21, 2024
@smelc smelc removed the Stale label Aug 5, 2024
Copy link

github-actions bot commented Sep 5, 2024

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Sep 5, 2024
@smelc smelc removed the Stale label Sep 5, 2024
Copy link

github-actions bot commented Oct 6, 2024

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Oct 6, 2024
@smelc smelc removed the Stale label Oct 7, 2024
Copy link

github-actions bot commented Nov 7, 2024

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Nov 7, 2024
@smelc smelc removed the Stale label Nov 7, 2024
Copy link

github-actions bot commented Dec 8, 2024

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Dec 8, 2024
@smelc smelc removed the Stale label Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants