Skip to content

feat(repo): github profile url should be valid#268

Merged
Berkmann18 merged 2 commits intoall-contributors:masterfrom
k3nsei:feature/validate-profile-url
Jun 7, 2020
Merged

feat(repo): github profile url should be valid#268
Berkmann18 merged 2 commits intoall-contributors:masterfrom
k3nsei:feature/validate-profile-url

Conversation

@k3nsei
Copy link
Copy Markdown
Contributor

@k3nsei k3nsei commented Jun 7, 2020

What:
Contributor profile url should be valid url

Why:
Please see issue #267

How:
Validator was added and used where github profile url is assigned

Checklist:

  • [N/A] Documentation
  • Tests
  • Ready to be merged
  • Added myself to contributors table

@k3nsei
Copy link
Copy Markdown
Contributor Author

k3nsei commented Jun 7, 2020

@Berkmann18 please do code review

@Berkmann18 Berkmann18 linked an issue Jun 7, 2020 that may be closed by this pull request
@Berkmann18 Berkmann18 self-assigned this Jun 7, 2020
Copy link
Copy Markdown
Member

@Berkmann18 Berkmann18 left a comment

Choose a reason for hiding this comment

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

That looks good to me. Have you tried that on your repo where you originally had the issue?

@k3nsei
Copy link
Copy Markdown
Contributor Author

k3nsei commented Jun 7, 2020

yarn run all-contributors add Silvest89 bug
yarn run v1.22.4
$ /app/projects/ng-in-viewport/node_modules/.bin/all-contributors add Silvest89 bug
Cannot read property 'isValidHttpUrl' of undefined
error Command failed with exit code 1.

Don't know why const util = require('../util') doesn't work and returns empty object.

@k3nsei
Copy link
Copy Markdown
Contributor Author

k3nsei commented Jun 7, 2020

@Berkmann18 I changed import from const util = require('../util') to const utilUrl = require('../util/url') and now everything works as expected.

@k3nsei
Copy link
Copy Markdown
Contributor Author

k3nsei commented Jun 7, 2020

@Berkmann18 now all should be ok

@Berkmann18 Berkmann18 merged commit b77f86b into all-contributors:master Jun 7, 2020
@all-contributors-release-bot
Copy link
Copy Markdown
Member

🎉 This PR is included in version 6.16.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@k3nsei k3nsei deleted the feature/validate-profile-url branch June 7, 2020 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Contributor profile url need to be validated

3 participants