Skip to content
This repository has been archived by the owner on Aug 20, 2020. It is now read-only.

chore: Adds test to check license end year. #56

Merged
merged 1 commit into from
Nov 11, 2017

Conversation

mukulmishra18
Copy link
Contributor

Fixes #51

Please note that added test is failing right now, as LICENSE.txt file does not have correct end year. Also this test expects the year in range(i.e. 2016-2017), but only contains 2016. Maybe fix after #53

Copy link
Contributor

@mbernier mbernier left a comment

Choose a reason for hiding this comment

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

hey @mukulmishra18, I got the license file updated in another PR.
Now that we have see that your code can catch it when the year is wrong, can you please merge from master and let's see if the test passes with the year being correct!

@mukulmishra18
Copy link
Contributor Author

Hi @mbernier, I have updated the PR to new master and it is passing LICENSE test locally, Thanks.

@mbernier
Copy link
Contributor

mbernier commented Nov 1, 2017

There was a merge conflict and an extra parenthesis on line 321, looked like this:

  function fileExists(filepath) {
    try {
      return fs.statSync(filepath).isFile();
    } catch(e) {
      if (e.code === 'ENOENT') {
        console.log('' + filepath + ' doesn\'t exist.');
        return false;
      }
      throw e;
    }
  })
})

it is now this:

  function fileExists(filepath) {
    try {
      return fs.statSync(filepath).isFile();
    } catch(e) {
      if (e.code === 'ENOENT') {
        console.log('' + filepath + ' doesn\'t exist.');
        return false;
      }
      throw e;
    }
  } //no paren here any longer
})

@thinkingserious
Copy link
Contributor

@flaredragon

We have not been able to merge your Pull Request, but because you are awesome - we wanted to make sure you could still get a SendGrid Hacktoberfest shirt.

Please go fill out our swag form before Nov 5th and we will send the shirt! (We know that you might have tried this before and it didn’t work, sorry about that!)

You have till Nov 5th to fill out this form in order to get the Hacktoberfest shirt!

Thank you for contributing during Hacktoberfest! We hope to see you in the repos soon! Just so you know, we always give away a SendGrid shirt for your first ever non-Hacktoberfest PR that gets merged.

@flaredragon
Copy link
Contributor

flaredragon commented Nov 2, 2017

@thinkingserious Thanks a Lot!
My PR in another Sendgrid repository was merged during Hacktoberfest So I have filled the form before.
Will be waiting for Tshirt!
Regards.

@mukulmishra18
Copy link
Contributor Author

I think it should be fine now. @mbernier can you please have a look?

@thinkingserious
Copy link
Contributor

thinkingserious commented Nov 3, 2017

Thanks @mukulmishra18!

Your PR is now scheduled for review :) Please bear with us as we have fallen behind on our reviews due to the 1,000+ PRs in October.

Big thanks for your contribution and patience.

With Best Regards,

Elmer

@mbernier mbernier merged commit 59afed3 into sendgrid:master Nov 11, 2017
@thinkingserious
Copy link
Contributor

Hello @mukulmishra18,

Thanks again for the PR!

We appreciate your contribution and look forward to continued collaboration. Thanks!

Team SendGrid DX

@mbernier
Copy link
Contributor

If you are interested, we would love the opportunity to talk to you about Hacktoberfest and our API libraries.

Our agenda would be: Explore what you liked and is there anything we can do to improve?

You can grab a time on my calendar that works for you and we can have a chat on Google Hangout or Skype. If you prefer, you can email me using my GitHub username at my company’s domain.

Thank you so much,

Matt Bernier - @mbernier - SendGrid Developer Experience Product Manager
Elmer Thomas - @thinkingserious - SendGrid Developer Experience Engineer
@sendgrid

@thinkingserious thinkingserious changed the title Adds test to check license end year. chore: Adds test to check license end year. Dec 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a unittest to check the license.md file date range
4 participants