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

Proposal Linting Proof of Concept #36

Closed
wants to merge 4 commits into from

Conversation

jamesrp13
Copy link
Contributor

@jacobcreech here's a proof of concept for linting. Haven't gotten around to any kind of refactoring and testing or adding it as a Github workflow, but if you npm run lint locally on this you'll see that it runs the linter on everything in the proposals directory and shows errors for not matching header structure and frontmatter. Not where I want it to be but maybe an okay starting place.

@jamesrp13 jamesrp13 changed the title Proposal Listing Proof of Concept Proposal Linting Proof of Concept Feb 10, 2023
Copy link

@hydrogenbond007 hydrogenbond007 left a comment

Choose a reason for hiding this comment

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

hey @jamesrp13 , do you have any md file explaining it a little bit(wanted to learn more about it)? btw trent recommended working on the proposal before the code #34

@jacobcreech
Copy link
Contributor

@hydrogenbond007 this is not a proposal but an attempt to automate some of the janitorial duties. I'm building off this to create ci for some of the basic proposal checks

@hydrogenbond007
Copy link

oh gotcha 😅

@jacobcreech
Copy link
Contributor

Closing to push in a separate PR

@jacobcreech jacobcreech closed this Mar 8, 2023
@jacobcreech jacobcreech mentioned this pull request Mar 8, 2023
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.

3 participants