Skip to content

markdown checker - #226

Draft
oyilmaztekin wants to merge 36 commits into
tc39:mainfrom
oyilmaztekin:ozer-check-markdown
Draft

markdown checker#226
oyilmaztekin wants to merge 36 commits into
tc39:mainfrom
oyilmaztekin:ozer-check-markdown

Conversation

@oyilmaztekin

Copy link
Copy Markdown
Contributor

#140

  • added markdown reader
  • parsed markdown to the AST
  • detected all tables
  • collected all link definitions
  • started to add tests

Discussions: #140 (comment)

added parser

detected all tables into the MD file

collected all link definitions

started to add tests
Comment thread package.json Outdated
Comment thread package.json Outdated
Comment thread tools/markdown-checker/package.json
changed pacakge publication to private
Comment thread tools/markdown-checker/lib/analyzer/collectLinkDefinitions.js Outdated
Comment thread tools/markdown-checker/lib/analyzer/detectHeaders.js Outdated
Comment thread tools/markdown-checker/lib/analyzer/detectHeaders.js Outdated
Comment thread tools/markdown-checker/lib/analyzer/detectTables.js Outdated
Comment thread tools/markdown-checker/package.json Outdated
Comment thread tools/markdown-checker/lib/transformer/traverser.js Outdated
Comment thread tools/markdown-checker/lib/parser/__tests__/readMarkdown.test.js Outdated
Comment thread tools/markdown-checker/lib/parser/__tests__/readMarkdown.test.js Outdated
Comment thread tools/markdown-checker/lib/parser/__tests__/readMarkdown.test.js Outdated
Comment thread tools/markdown-checker/lib/analyzer/analyzeTable.js Outdated
@oguzzkilic

Copy link
Copy Markdown

Adding it in eslint + airbnb configuration can be good for code standard.

oyilmaztekin and others added 10 commits July 3, 2019 00:30
Co-Authored-By: Jordan Harband <ljharb@gmail.com>
Co-Authored-By: Jordan Harband <ljharb@gmail.com>
Co-Authored-By: Jordan Harband <ljharb@gmail.com>
Co-Authored-By: Jordan Harband <ljharb@gmail.com>
Co-Authored-By: Jordan Harband <ljharb@gmail.com>
Co-Authored-By: Jordan Harband <ljharb@gmail.com>
Co-Authored-By: Jordan Harband <ljharb@gmail.com>
Co-Authored-By: Jordan Harband <ljharb@gmail.com>
added missing newlines
Comment thread tools/markdown-checker/index.js Outdated
Comment thread tools/markdown-checker/lib/analyzer/analyzeTable.js Outdated
Comment thread tools/markdown-checker/lib/transformer/traverser.js Outdated
@septs

septs commented Sep 27, 2020

Copy link
Copy Markdown
Member

@ljharb

  1. I want to integrate https://github.com/DavidAnson/markdownlint in gh-actions on push and pull-request pre-check

  2. I want to integrate a link checker that is available every day

    1. check is redirected (3xx status code)
      e.q: repo transferred or renamed

    2. check is not found (404 status code)
      e.q: repo deleted

    3. don't check not github

  3. Add Visual Studio Code pre-settings

    1. .vscode/settings.json
    {
        "editor.formatOnSave": true, // auto format
        "editor.codeActionsOnSave": {
        "source.fixAll": true // auto fix
     }
    }
    
    1. .vscode/extensions.json

    prompt to install extensions

    {
      "recommendations": ["DavidAnson.vscode-markdownlint"]
    }
    

@Jack-Works

Copy link
Copy Markdown
Member

Not everyone using VSCode.

@ljharb

ljharb commented Sep 28, 2020

Copy link
Copy Markdown
Member

Definitely editor-specific stuff isn't a good idea to include. If it's a linter tho, it should be able to run locally via eslint, which would be fine.

The trick with an action (or any CI check) is that inevitably someone's going to be blocked by a link going dark when it's unrelated to the change they're making. What we really need is something that can notice a link is dead, and then open up a PR or issue about it.

@septs

septs commented Sep 28, 2020

Copy link
Copy Markdown
Member

@ljharb i think, i can integrate one link-checker on github action

on push and scheduled (run once a day)

@ljharb

ljharb commented Sep 28, 2020

Copy link
Copy Markdown
Member

@septs how would we be notified when it failed? would that cause PRs to suddenly be blocked?

@septs

septs commented Sep 28, 2020

Copy link
Copy Markdown
Member

@ljharb ye, will a notification, but will not prevent PR.

@septs

septs commented Sep 28, 2020

Copy link
Copy Markdown
Member

@septs

septs commented Sep 28, 2020

Copy link
Copy Markdown
Member

Comment thread .idea/.gitignore Outdated
Comment thread .gitignore Outdated
@oyilmaztekin

oyilmaztekin commented Sep 11, 2021

Copy link
Copy Markdown
Contributor Author

@ljharb @littledan @chicoxyzzy final JSON output was created for stage 1. Still, things need to be improved, but let's discuss the output json file that has been added to the commit. This output was created automatically. Also, you can share your concerns about deployment pipeline implementation and the code quality.

For refreshing memories of what this PR was about
#140 (comment)
#188

@oyilmaztekin

Copy link
Copy Markdown
Contributor Author

In addition, you can review the JSON output here. It will be the final result after we are done with that PR

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.

7 participants