Skip to content

Conversation

@amolkahat
Copy link
Member

  • It will check link is valid or not.
  • If link is like https://<hostname>:<port> then it should mark
    it as skip.

Signed-off-by: Amol Kahat amolkahat@gmail.com

Fixes #12

@amolkahat amolkahat force-pushed the url_validator branch 2 times, most recently from 25ec6d8 to e28a60f Compare December 7, 2019 05:41
@amolkahat amolkahat requested a review from digitronik December 7, 2019 05:41
@raukadah
Copy link
Contributor

raukadah commented Dec 7, 2019

Please add some test data in tests folder so that it can be validated

@digitronik
Copy link
Member

@amolkahat thanks for PR... please have look on pre-commit checks for more info Analysis (git diff) block.

@amolkahat amolkahat force-pushed the url_validator branch 3 times, most recently from 78b5df7 to 97f5609 Compare December 7, 2019 09:06
- It will check link is valid or not.
- If link is like `https://<hostname>:<port>` then it should mark
it as skip.

Signed-off-by: Amol Kahat <amolkahat@gmail.com>
Copy link
Member

@digitronik digitronik left a comment

Choose a reason for hiding this comment

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

thanks for pr

@raukadah raukadah merged commit 1ba75ef into pythonpune:master Dec 7, 2019
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.

Need validator to check links first then check for the reachout.

3 participants