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

Patch 16 with travis integration fixes #593

Open
wants to merge 70 commits into
base: master
Choose a base branch
from

Conversation

copolycube
Copy link
Contributor

@copolycube copolycube commented Oct 1, 2020

Fixed the travis integration so that it works again
this lead to some error detections in the checked names, which I also corrected to make them all compliants.

copolycube and others added 30 commits October 30, 2017 21:42
French translation.
added french version and another alias (s/i/y)
added the french translation for the male version.
NB : There is also a female version in french for this first name, (andrée) how should it be added ?
added french translation
note : also submitting the feminin version of this since it does exist with an e at the end for femals.
french translation
added the female first name "Solange"
moving the file into "collection/" directory
new first name
new indian first name added
Bumps [morgan](https://github.com/expressjs/morgan) from 1.9.0 to 1.9.1.
- [Release notes](https://github.com/expressjs/morgan/releases)
- [Changelog](https://github.com/expressjs/morgan/blob/master/HISTORY.md)
- [Commits](expressjs/morgan@1.9.0...1.9.1)

Signed-off-by: dependabot[bot] <support@github.com>
@TravisBuddy
Copy link

Build failed

Hey Copolycube,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

Node.js: 8

npm test
> name-db@1.0.0 test /home/travis/build/bluzi/name-db
> mocha



  name files
    1) should contain valid JSON
    2) should contain a lowercase name, same as the filename
    3) should not have duplicate names
    4) should have ISO-639-3 language codes
    5) should have correct structure

  Translations
    6) should be lowercase

  Sex
    7) should be "m", "f" or "u"
    8) should be lowercase


  0 passing (17ms)
  8 failing

  1) name files
       should contain valid JSON:
     Error: Error in name file "abigael.json": Invalid JSON file 
SyntaxError: Unexpected token . in JSON at position 4
      at Context.it (test/test.js:16:23)

  2) name files
       should contain a lowercase name, same as the filename:
     SyntaxError: Unexpected token . in JSON at position 4
      at JSON.parse (<anonymous>)
      at Context.it (test/test.js:24:31)

  3) name files
       should not have duplicate names:
     SyntaxError: Unexpected token . in JSON at position 4
      at JSON.parse (<anonymous>)
      at Context.it (test/test.js:35:31)

  4) name files
       should have ISO-639-3 language codes:
     SyntaxError: Unexpected token . in JSON at position 4
      at JSON.parse (<anonymous>)
      at Context.it (test/test.js:47:31)

  5) name files
       should have correct structure:
     Error: Error in name file "abigael.json":
SyntaxError: Unexpected token . in JSON at position 4
      at Context.it (test/test.js:106:23)

  6) Translations
       should be lowercase:
     SyntaxError: Unexpected token . in JSON at position 4
      at JSON.parse (<anonymous>)
      at Context.it (test/test.js:116:31)

  7) Sex
       should be "m", "f" or "u":
     SyntaxError: Unexpected token . in JSON at position 4
      at JSON.parse (<anonymous>)
      at Context.it (test/test.js:131:31)

  8) Sex
       should be lowercase:
     SyntaxError: Unexpected token . in JSON at position 4
      at JSON.parse (<anonymous>)
      at Context.it (test/test.js:145:31)



npm ERR! Test failed.  See above for more details.

@TravisBuddy
Copy link

Build failed

Hey Copolycube,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

Node.js: 8

npm test
> name-db@1.0.0 test /home/travis/build/bluzi/name-db
> mocha



  name files
    1) should contain valid JSON
    2) should contain a lowercase name, same as the filename
    3) should not have duplicate names
    4) should have ISO-639-3 language codes
    5) should have correct structure

  Translations
    6) should be lowercase

  Sex
    7) should be "m", "f" or "u"
    8) should be lowercase


  0 passing (176ms)
  8 failing

  1) name files
       should contain valid JSON:
     Error: Error in name file "sai.json": Invalid JSON file 
SyntaxError: Unexpected string in JSON at position 250
      at Context.it (test/test.js:16:23)

  2) name files
       should contain a lowercase name, same as the filename:
     SyntaxError: Unexpected string in JSON at position 250
      at JSON.parse (<anonymous>)
      at Context.it (test/test.js:24:31)

  3) name files
       should not have duplicate names:
     SyntaxError: Unexpected string in JSON at position 250
      at JSON.parse (<anonymous>)
      at Context.it (test/test.js:35:31)

  4) name files
       should have ISO-639-3 language codes:
     SyntaxError: Unexpected string in JSON at position 250
      at JSON.parse (<anonymous>)
      at Context.it (test/test.js:47:31)

  5) name files
       should have correct structure:
     Error: Error in name file "sai.json":
SyntaxError: Unexpected string in JSON at position 250
      at Context.it (test/test.js:106:23)

  6) Translations
       should be lowercase:
     SyntaxError: Unexpected string in JSON at position 250
      at JSON.parse (<anonymous>)
      at Context.it (test/test.js:116:31)

  7) Sex
       should be "m", "f" or "u":
     SyntaxError: Unexpected string in JSON at position 250
      at JSON.parse (<anonymous>)
      at Context.it (test/test.js:131:31)

  8) Sex
       should be lowercase:
     SyntaxError: Unexpected string in JSON at position 250
      at JSON.parse (<anonymous>)
      at Context.it (test/test.js:145:31)



npm ERR! Test failed.  See above for more details.

@copolycube
Copy link
Contributor Author

Hello @bluzi are you still maintaining this project ?

@copolycube
Copy link
Contributor Author

@bluzi Hello, this PR should allow the CI on this project to be re-activated correctly :-)

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.

4 participants