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

Add missing frontmatter, including stages #835

Merged
merged 6 commits into from
Aug 9, 2022
Merged

Conversation

wagenet
Copy link
Member

@wagenet wagenet commented Aug 3, 2022

This includes #804.

@wagenet wagenet force-pushed the missing-frontmatter branch from ee6525a to 0a52d1b Compare August 4, 2022 17:51
@wagenet wagenet changed the title WIP: Add missing frontmatter Add missing frontmatter Aug 4, 2022
@wagenet wagenet marked this pull request as ready for review August 4, 2022 17:51
@wagenet wagenet changed the title Add missing frontmatter Add missing frontmatter, including stages Aug 4, 2022
@wagenet wagenet force-pushed the missing-frontmatter branch from 690fec5 to c5d447a Compare August 5, 2022 18:38
@@ -1,10 +1,19 @@
---
stage: accepted
stage: discontinued # FIXME: Is this correct?
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could also leave this as recommended. Not quite clear how we want to handle this sort of thing.

@@ -1,10 +1,10 @@
---
stage: accepted
stage: discontinued # FIXME: Is this correct?
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again, not sure whether this should be Discontinued or remain in Recommended.

@wagenet wagenet force-pushed the missing-frontmatter branch from 8cdd20d to bc6ebca Compare August 8, 2022 21:34
@wagenet wagenet merged commit 5515aa4 into master Aug 9, 2022
@delete-merged-branch delete-merged-branch bot deleted the missing-frontmatter branch August 9, 2022 04:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants