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

Gitignore #1450

Closed
wants to merge 2 commits into from
Closed

Gitignore #1450

wants to merge 2 commits into from

Conversation

rubiin
Copy link
Member

@rubiin rubiin commented Sep 30, 2020

Added few folders that contains generated code. Thus the git status does not get messy

Checklist

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

@codecov
Copy link

codecov bot commented Sep 30, 2020

Codecov Report

Merging #1450 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1450   +/-   ##
=======================================
  Coverage   99.92%   99.92%           
=======================================
  Files          95       95           
  Lines        1268     1270    +2     
=======================================
+ Hits         1267     1269    +2     
  Misses          1        1           
Impacted Files Coverage Δ
src/lib/isEmail.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 685c3d2...fc66787. Read the comment docs.

@rubiin
Copy link
Member Author

rubiin commented Sep 30, 2020

@profnandaa check this only after my previous pr

@@ -5,3 +5,8 @@ coverage.lcov
.nyc_output
package-lock.json
yarn.lock
es
Copy link
Member

Choose a reason for hiding this comment

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

What is your opinion in ignoring the generated files @profnandaa and @tux-tn?

@@ -101,7 +101,7 @@ Validator | Description
**isDecimal(str [, options])** | check if the string represents a decimal number, such as 0.1, .3, 1.1, 1.00003, 4.0, etc.<br/><br/>`options` is an object which defaults to `{force_decimal: false, decimal_digits: '1,', locale: 'en-US'}`<br/><br/>`locale` determine the decimal separator and is one of `['ar', 'ar-AE', 'ar-BH', 'ar-DZ', 'ar-EG', 'ar-IQ', 'ar-JO', 'ar-KW', 'ar-LB', 'ar-LY', 'ar-MA', 'ar-QA', 'ar-QM', 'ar-SA', 'ar-SD', 'ar-SY', 'ar-TN', 'ar-YE', 'bg-BG', 'cs-CZ', 'da-DK', 'de-DE', 'en-AU', 'en-GB', 'en-HK', 'en-IN', 'en-NZ', 'en-US', 'en-ZA', 'en-ZM', 'es-ES', 'fr-FR', 'hu-HU', 'it-IT', 'ku-IQ', nb-NO', 'nl-NL', 'nn-NO', 'pl-PL', 'pt-BR', 'pt-PT', 'ru-RU', 'sl-SI', 'sr-RS', 'sr-RS@latin', 'sv-SE', 'tr-TR', 'uk-UA']`.<br/>**Note:** `decimal_digits` is given as a range like '1,3', a specific value like '3' or min like '1,'.
**isDivisibleBy(str, number)** | check if the string is a number that's divisible by another.
**isEAN(str)** | check if the string is an EAN (European Article Number).
**isEmail(str [, options])** | check if the string is an email.<br/><br/>`options` is an object which defaults to `{ allow_display_name: false, require_display_name: false, allow_utf8_local_part: true, require_tld: true, allow_ip_domain: false, domain_specific_validation: false }`. If `allow_display_name` is set to true, the validator will also match `Display Name <email-address>`. If `require_display_name` is set to true, the validator will reject strings without the format `Display Name <email-address>`. If `allow_utf8_local_part` is set to false, the validator will not allow any non-English UTF8 character in email address' local part. If `require_tld` is set to false, e-mail addresses without having TLD in their domain will also be matched. If `ignore_max_length` is set to true, the validator will not check for the standard max length of an email. If `allow_ip_domain` is set to true, the validator will allow IP addresses in the host part. If `domain_specific_validation` is true, some additional validation will be enabled, e.g. disallowing certain syntactically valid email addresses that are rejected by GMail.
**isEmail(str [, options])** | check if the string is an email.<br/><br/>`options` is an object which defaults to `{ allow_display_name: false, require_display_name: false, allow_utf8_local_part: true, require_tld: true, allow_ip_domain: false, domain_specific_validation: false, blacklisted_chars: '' }`. If `allow_display_name` is set to true, the validator will also match `Display Name <email-address>`. If `require_display_name` is set to true, the validator will reject strings without the format `Display Name <email-address>`. If `allow_utf8_local_part` is set to false, the validator will not allow any non-English UTF8 character in email address' local part. If `require_tld` is set to false, e-mail addresses without having TLD in their domain will also be matched. If `ignore_max_length` is set to true, the validator will not check for the standard max length of an email. If `allow_ip_domain` is set to true, the validator will allow IP addresses in the host part. If `domain_specific_validation` is true, some additional validation will be enabled, e.g. disallowing certain syntactically valid email addresses that are rejected by GMail. If `blacklisted_chars` recieves a string,then the validator will reject emails that include any of the characters in the string, in the name part.
Copy link
Member

Choose a reason for hiding this comment

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

These are the same files changed as in #1449

@@ -12,6 +12,13 @@ function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o =

function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) { arr2[i] = arr[i]; } return arr2; }

import merge from './util/merge';
Copy link
Member

Choose a reason for hiding this comment

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

There are many changes here that are the same as in #1449.
Please remove them

@rubiin
Copy link
Member Author

rubiin commented Sep 30, 2020

i am closing this as and will do another pr

@rubiin rubiin closed this Sep 30, 2020
@rubiin rubiin deleted the gitignore branch September 30, 2020 17:43
@profnandaa
Copy link
Member

Remember to link it to the issue once done.

profnandaa added a commit that referenced this pull request Nov 24, 2020
We are removing validator.js and validator.min.js since they are among
the top files that cause unnecessary conflicts, yet they are dependent
on the build system.

this is a follow up of #1450
profnandaa added a commit that referenced this pull request Nov 24, 2020
We are removing validator.js and validator.min.js since they are among
the top files that cause unnecessary conflicts, yet they are dependent
on the build system.

this is a follow up of #1450
profnandaa added a commit that referenced this pull request Nov 24, 2020
We are removing validator.js and validator.min.js since they are among
the top files that cause unnecessary conflicts, yet they are dependent
on the build system.

this is a follow up of #1450
profnandaa added a commit that referenced this pull request Nov 24, 2020
We are removing validator.js and validator.min.js since they are among
the top files that cause unnecessary conflicts, yet they are dependent
on the build system.

this is a follow up of #1450
profnandaa added a commit that referenced this pull request Nov 24, 2020
We are removing validator.js and validator.min.js since they are among
the top files that cause unnecessary conflicts, yet they are dependent
on the build system.

this is a follow up of #1450
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.

3 participants