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

Fixed markdown lints #70

Merged
merged 2 commits into from
Jan 2, 2024
Merged

Conversation

BenMatase
Copy link

So that we no longer have to ignore the markdown lint errors

Copy link
Collaborator

@gregschrock gregschrock left a comment

Choose a reason for hiding this comment

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

This is fine; it might make merges from remote a bit of a pain though.

@BenMatase
Copy link
Author

BenMatase commented Dec 19, 2023

This is fine; it might make merges from remote a bit of a pain though.

Sure. I mean the other thing is that I could fix our files and then skip linting (assuming we can do that) for the common merge conflict files. Specifically README.md at the top level

@gregschrock
Copy link
Collaborator

That's not a bad idea. I don't have a strong preference either way.

@BenMatase
Copy link
Author

Undid the testing changes. Lets move forward with what I have/had

@BenMatase BenMatase merged commit 14e36d6 into release-128tech-1.22 Jan 2, 2024
5 checks passed
@BenMatase BenMatase deleted the PLUGIN-2282-fix-lint-errors branch January 2, 2024 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants