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

Setup github actions #1553

Merged
merged 19 commits into from
Feb 15, 2021
Merged

Setup github actions #1553

merged 19 commits into from
Feb 15, 2021

Conversation

tux-tn
Copy link
Member

@tux-tn tux-tn commented Dec 12, 2020

Migrated from Travis-ci to github action for CI jobs

The main reason is that travis-ci.org is closing on 31 December and travis-ci.com have limits for open source projects. For more info you can check this article .

CI pipelines are triggered by new pull requests and pushes to master branch. Tests are run on multiple Node.js versions (14, 12, 10, 8) and coverage is sent to codecov after the tests succeed (only one time in the Node.js 14 pipeline).

You can find below example of action pipeline:

Setup automatic npm publishing

As discussed in #1540 , i created a new github action to publish a new version to npm registry on new release (the publishing happens only when a release is created). The main goal is to prevent having missing files or broken versions published.

In order to test this feature, i created a scoped package @validatorjs/validator-dev (Don't use this package, it's for demo purpose only).

@profnandaa The only change that we need to make is to create a new secret named NPM_SECRET containing as the name may suggest a generated npm access token (The type need to be automation to bypass 2 Factor Auth) with write access to the package.

Other changes

  • Deleted validator.js and validator.min.js auto-generated files
  • Replaced chriso/validator.js with validatorjs/validator.js in all urls to prevent unnecessary HTTP redirect
  • Fixed an issue introduced by Multiple fixes related to alpha, tests and build #1528 where src/index.js would be ignored by .gitignore
  • Replaced badges in Readme file

Checklist

  • PR contains only changes related; no stray files, etc.
  • README updated (where applicable)

@profnandaa
Copy link
Member

@tux-tn -- thanks so much for such a thorough job on this one! So that we do testing with pseudo-releases, I have created a temp branch, releases, you can change the base of the PR to it then we will later on move everything to master.

@profnandaa
Copy link
Member

/cc. @chriso

@tux-tn tux-tn changed the base branch from master to releases December 13, 2020 14:30
@tux-tn
Copy link
Member Author

tux-tn commented Dec 13, 2020

@profnandaa my pleasure! base branch has been changed to releases

Ashutosh Kumar and others added 4 commits December 17, 2020 07:21
…rjs#1554)

* fix(isMobilePhone): update de-CH locale (validatorjs#1549)
feat(isMobilePhone): add fr-CH locale (validatorjs#1549)
feat(isMobilePhone): add it-CH locale (validatorjs#1549)

* fix(isMobilePhone): update de-CH, fr-CH and it-CH locale in validator.min.js(validatorjs#1549)

* fix(isMobilePhone): update fr-CH and it-CH locale aliases (validatorjs#1549)

Co-authored-by: Ashutosh Kumar <[email protected]>
* Added isLicensePlate check with german locale

* de-LI (Liechtenstein) validator

* Added albanian number plates

* reversed changes in index.js

* reversed changes in index.js

* Refactored de-DE into one big regex, expanded tests fot de-DE

* added pt-PT locale

* Fixed coverage problem

* feat(isLicensePlate): clean build

* coverage is now 100%

* Update README.md

* Update README.md
Add correct tax identifier to demonstrate issue and fix.

Thanks to: Panos Papadopoulos
…lidatorjs#1575)

* Support new VN mobile phone 087 (https://didong.itelecom.vn/)

* Add test for new telco number

Co-authored-by: Kyle Dinh <[email protected]>
branches: [master]
jobs:
test:
runs-on: ubuntu-latest
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we use some specific version instead of latest for stability?

Copy link
Member Author

Choose a reason for hiding this comment

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

latest tag is for the latest Ubuntu LTS version but i agree with you, i will set it to ubuntu-20.04

Copy link
Member

@profnandaa profnandaa left a comment

Choose a reason for hiding this comment

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

Sorry for my awefully long delay on this @tux-tn ; This LGTM. Thanks for the work!

@profnandaa profnandaa merged commit 9debd5e into validatorjs:releases Feb 15, 2021
@profnandaa
Copy link
Member

@tux-tn -- sorry missed on this, could you do the PR against master?

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.

6 participants